Skip to content
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

feat(mobile): partner sharing #2541

Merged
merged 8 commits into from
May 25, 2023
Merged

feat(mobile): partner sharing #2541

merged 8 commits into from
May 25, 2023

Conversation

fyfrey
Copy link
Contributor

@fyfrey fyfrey commented May 23, 2023

Note: server version and app version must match for the partner sharing to work in the app

implements partner sharing functionality in the app.

  • manage(add/remove) partner
  • view partner assets
  • sync partner assets to local database
  • added userId parameter to AssetSearchDto for getAllAssets route on the server
  • modify syncing operations to account for different owners when deleting shared albums
  • reworked asset provider to account for different users to display assets for
  • enrich local User class with partner attributes
  • add local ETag database entity to store different eTags for different requests (so far getAllAssets for multiple users) to cache these large requests

some screenshots below:

image
image
image
image
image
image

@vercel
Copy link

vercel bot commented May 23, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
immich ⬜️ Ignored (Inspect) May 24, 2023 10:51am

@fyfrey fyfrey requested review from alextran1502 and brighteyed May 23, 2023 16:42
Copy link
Contributor

@michelheusschen michelheusschen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested this in profile mode if that might matter. Ran into some syncing issues, leading to an empty partner page. Logs showed _linkWithExistingFromDb encountered asset only in DB

On the main timeline there was also an issue with assets showing up twice on regular load and even 3 or 4 times when I manually refresh (see video below). Logs for regular load:

#0 Load assets: 243ms
#1 Upserted 59 assets into the DB
#2: Upserted 1 assets into the DB
#3: Upserted 59 assets into the DB
2023-05-24.09-20-12.mp4

@alextran1502 alextran1502 merged commit bcc2c34 into main May 25, 2023
@alextran1502 alextran1502 deleted the dev/mobile-partner-sharing branch May 25, 2023 03:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants