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

chore(mobile): add toast notification for "Download Started" #9023

Merged
merged 1 commit into from
Apr 26, 2024

Conversation

DeadEnded
Copy link
Contributor

Recently started self hosting Immich and using the mobile Android app (fantastic work being done by the way!). We recently noticed that when we started a download in the app - nothing happened. Downloading large files (long videos) gave no feedback until the download was completed.

So, I have made an attempt to add a toast notification at the start of the download to signal that the download did indeed start so the user has confirmation that they initiated the download. I have Google Translated as many of the languages as I felt comfortable with the results of. If I should remove these and let a native speaker do their languages translations (so nothing is mistranslated into something inappropriate) please let me know.

I am a novice, but very much appreciate the projects I use and contribute when I can. Please let me know if anything needs changed/fixed/removed, etc. and I'm happy to do the work!

Thank you - Cheers!
DeadEnd

@alextran1502
Copy link
Contributor

Thank you for your contribution, you will only need to add the translation string to en-US.json file for future reference

@alextran1502 alextran1502 changed the title Add Mobile toast notification for "Download Started" chore(mobile): add toast notification for "Download Started" Apr 23, 2024
@alextran1502 alextran1502 enabled auto-merge (squash) April 23, 2024 13:11
Add Toast notification for mobile download started.
Added new placeholder in each language file - populated with best estimate translations.
auto-merge was automatically disabled April 24, 2024 03:00

Head branch was pushed to by a user without write access

@alextran1502 alextran1502 merged commit 00d186e into immich-app:main Apr 26, 2024
21 of 22 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.

None yet

2 participants