Skip to content

fix: process query param for my.immich.app/memory?id=#25474

Merged
alextran1502 merged 3 commits intoimmich-app:mainfrom
arne182:patch-1
Jan 25, 2026
Merged

fix: process query param for my.immich.app/memory?id=#25474
alextran1502 merged 3 commits intoimmich-app:mainfrom
arne182:patch-1

Conversation

@arne182
Copy link
Contributor

@arne182 arne182 commented Jan 23, 2026

Add regex for memory deep link handling of
https://my.immich.app/memory?id=
Completing the work for the Android app to handle memory links. #25373

Description

This is a continuation of this PR to be able to handle the https://my.immich.app/memory?id=
#25373

Fixes # (issue)

How Has This Been Tested?

  • No Tests done

Screenshots (if appropriate)

Checklist:

  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation if applicable
  • I have no unrelated changes in the PR.
  • I have confirmed that any new dependencies are strictly necessary.
  • I have written tests for new code (if applicable)
  • I have followed naming conventions/patterns in the surrounding code
  • All code in src/services/ uses repositories implementations for database calls, filesystem operations, etc.
  • All code in src/repositories/ is pretty basic/simple and does not have any immich specific logic (that belongs in src/services/)

Please describe to which degree, if any, an LLM was used in creating this pull request.

Code completly writen by LLM. Please help me review it @bwees
...

Add regex for memory deep link handling of
https://my.immich.app/memory?id=<uuid>
Completing the work for the Android app to handle memory links.
immich-app#25373
This still needs to be tested as it is AI suggested code.
@bwees bwees changed the title Implement memory deep link regex and logic fix: process query param for my.immich.app/memory?id= Jan 23, 2026
Copy link
Member

@bwees bwees left a comment

Choose a reason for hiding this comment

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

Need to wait for apple's CDN to grab the new site associations file.

https://app-site-association.cdn-apple.com/a/v1/my.immich.app

@arne182
Copy link
Contributor Author

arne182 commented Jan 25, 2026

@bwees thanks for the help on this PR

@alextran1502 alextran1502 merged commit 2ee903a into immich-app:main Jan 25, 2026
48 checks passed
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.

3 participants