Skip to content

fix(mobile): hide asset description text field if user is not owner #17442

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 2 commits into from
Apr 8, 2025

Conversation

gaganyadav80
Copy link
Contributor

  • If user is not the owner and asset has no description then hide the text field

Description

Fixes #7546

How Has This Been Tested?

  • When user is owner, the description text field is enabled and loads any existing content in the field.
  • When user is NOT the owner and asset already has description then field it is visible but disabled.
  • When user is neither owner nor the asset has description then field is hidden

* If user is not the owner and asset has no description then hide the text field
@alextran1502 alextran1502 merged commit 75bc32b into immich-app:main Apr 8, 2025
39 checks passed
savely-krasovsky pushed a commit to savely-krasovsky/immich that referenced this pull request Jun 8, 2025
…mmich-app#17442)

* fix(mobile): hide asset description text field if user is not owner

* If user is not the owner and asset has no description then hide the text field

* Apply suggestions from code review

Co-authored-by: Alex <[email protected]>

---------

Co-authored-by: Alex <[email protected]>
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.

Images in a shared album show the “add description” option
2 participants