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

fix(web): user list when sharing an album #6500

Merged
merged 2 commits into from
Jan 19, 2024

Conversation

martabal
Copy link
Member

Before After
2024-01-19.16-53-03.mp4
2024-01-19.16-52-09.mp4

Copy link

cloudflare-pages bot commented Jan 19, 2024

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: de8310a
Status:⚡️  Build in progress...

View logs

@@ -6221,7 +6221,7 @@
"info": {
"title": "Immich",
"description": "Immich API",
"version": "1.93.0",
"version": "1.93.1",
Copy link
Member

Choose a reason for hiding this comment

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

I don't think this PR should update this

Copy link
Member Author

Choose a reason for hiding this comment

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

It got updated automatically with git checkout main

>
<p>✓</p>
Copy link
Member

Choose a reason for hiding this comment

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

Why is there just an empty p tag?

Copy link
Member Author

Choose a reason for hiding this comment

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

It's not empty (✓)

@alextran1502
Copy link
Contributor

Can you help rebase the PR?

@alextran1502 alextran1502 enabled auto-merge (squash) January 19, 2024 17:36
@alextran1502 alextran1502 merged commit d15c443 into main Jan 19, 2024
19 of 21 checks passed
@alextran1502 alextran1502 deleted the fix/share-album-to-user-list branch January 19, 2024 17:38
@martabal
Copy link
Member Author

Can you help rebase the PR?

Sorry, you're too fast

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants