-
-
Notifications
You must be signed in to change notification settings - Fork 30
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
Migrate photo storage to GCloud storage #273
Conversation
@zichongkao @musoke @CocoisBuggy All frontend breaking changes are fixed in this PR: OpenBeta/open-tacos#800 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a huge set of changes! Looks like a more sustainable base for the future 😃
I did a dry-run import of production data locally and can confirm that existing tags were migrated to the new collection.
|
chore: resolve username in the backend instead of frontend chore: return image metadata to tag queries chore: rebase
refactoring
chore: climb/area/user profile media share the same type
cache external HTTP calls to recolve username
…se the new embedded tags
chore: set media upload time using real timestamp from sirv refactor: improve MediaDatasource's queries to use index
…ter performance and simplier queries
bug: fix how exception is passed to logger
chore: rename media GQL types & queries bug: fix latlng bug
* chore: extract & save image width/height during migration * chore: resolve username in the backend instead of frontend * chore: return image metadata to tag queries * chore: climb/area/user profile media share the same type * refactor: embed tags in media object collection * refactor: update find-tags-by-area-id to work with embedded tags * refactor: move leaderboard api to new embedded tags model
Issues:
TBD in a follow up PR:
Migration steps (I'll test this in staging first)
New data model
rawTags
is an embedded array of Tags eliminating the need to do alookup
(SQL join)