Skip to content

fix(Places): Use event metadata to compute the place #2966

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 30, 2025

Conversation

artonge
Copy link
Collaborator

@artonge artonge commented Apr 24, 2025

When using S3 bucket, all the metadata will be computed in a background job. This means that the GPS info won't be in the DB yet, but it will be in the event's metadata.

@artonge artonge self-assigned this Apr 24, 2025
@artonge artonge marked this pull request as ready for review April 24, 2025 16:08
@artonge artonge requested a review from ArtificialOwl April 24, 2025 16:08
@artonge artonge added bug Something isn't working 3. to review Waiting for reviews php PHP related ticket labels Apr 24, 2025
@artonge artonge added this to the Nextcloud 32 milestone Apr 24, 2025
@artonge
Copy link
Collaborator Author

artonge commented Apr 24, 2025

/backport to stable31

@artonge
Copy link
Collaborator Author

artonge commented Apr 24, 2025

/backport to stable30

@backportbot backportbot bot added the backport-request Pending backport by the backport-bot label Apr 24, 2025
@artonge artonge force-pushed the artonge/fix/placemapper_with_s3 branch 2 times, most recently from 951b8bf to 8f25ad2 Compare April 24, 2025 16:22
@artonge artonge force-pushed the artonge/chore/migrate_to_node_api branch from 25b10f2 to 9c4ada2 Compare April 24, 2025 16:30
@artonge artonge changed the base branch from artonge/chore/migrate_to_node_api to master April 24, 2025 16:30
When using S3 bucket, all the metadata will be computed in a background job. This means that the GPS info won't be in the DB yet, but it will be in the event's metadata.

Signed-off-by: Louis Chemineau <[email protected]>
@artonge artonge force-pushed the artonge/fix/placemapper_with_s3 branch from 8f25ad2 to 71401b3 Compare April 28, 2025 08:55
Copy link

codecov bot commented Apr 28, 2025

Codecov Report

Attention: Patch coverage is 0% with 837 lines in your changes missing coverage. Please review.

Project coverage is 2.28%. Comparing base (b815b68) to head (71401b3).
Report is 2 commits behind head on master.

Files with missing lines Patch % Lines
src/utils/fileUtils.ts 0.00% 51 Missing ⚠️
src/views/Folders.vue 0.00% 51 Missing ⚠️
src/mixins/FetchFacesMixin.ts 0.00% 47 Missing ⚠️
src/store/folders.ts 0.00% 45 Missing ⚠️
src/store/collections.ts 0.00% 44 Missing ⚠️
src/views/FaceContent.vue 0.00% 23 Missing ⚠️
src/views/PhotosTab.vue 0.00% 23 Missing ⚠️
src/components/Folder.vue 0.00% 22 Missing ⚠️
src/components/TagCover.vue 0.00% 22 Missing ⚠️
src/store/files.ts 0.00% 21 Missing ⚠️
... and 57 more
Additional details and impacted files
@@            Coverage Diff            @@
##           master   #2966      +/-   ##
=========================================
- Coverage    2.31%   2.28%   -0.03%     
=========================================
  Files          98      97       -1     
  Lines        6190    6215      +25     
  Branches      112     111       -1     
=========================================
- Hits          143     142       -1     
- Misses       6047    6073      +26     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@artonge artonge force-pushed the artonge/fix/placemapper_with_s3 branch from 71401b3 to a5951dc Compare April 28, 2025 09:06
@artonge artonge merged commit b33b948 into master Apr 30, 2025
48 checks passed
@artonge artonge deleted the artonge/fix/placemapper_with_s3 branch April 30, 2025 18:45
@backportbot backportbot bot removed the backport-request Pending backport by the backport-bot label Apr 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews bug Something isn't working php PHP related ticket
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants