Skip to content

feature/i18n#183

Merged
giejay merged 4 commits intogiejay:mainfrom
HappyKoalaDev:main
Feb 1, 2026
Merged

feature/i18n#183
giejay merged 4 commits intogiejay:mainfrom
HappyKoalaDev:main

Conversation

@HappyKoalaDev
Copy link
Copy Markdown
Contributor

@HappyKoalaDev HappyKoalaDev commented Jan 20, 2026

Added multilingual support.

  • Hardcoded strings moved to resource files.
  • Added Russian translation.

Closes #171

@giejay
Copy link
Copy Markdown
Owner

giejay commented Jan 20, 2026

This is great! I will review it somewhere this week

@@ -144,12 +145,12 @@ class ScreenSaverService : DreamService(), MediaSliderListener {
}
}
} else {
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Could you create a new resource string for this error? string.set_albums_screensaver_error or something?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Which line exactly are you referring to? I don't see any more untranslated strings in the ScreenSaverService.kt.

However, I found a few more hardcoded strings and added them to this PR as new commit.

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Previously, this was showing an error message with the text, please set the Immich albums to show in the settings (or something similar). Now it reuses the text from the settings itself, "set albums in screensaver"

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

thanks, fixed

@giejay
Copy link
Copy Markdown
Owner

giejay commented Jan 24, 2026

I see a change in the MetaDataItem getTitle function so I guess this also requires a PR to MediasliderView?

@HappyKoalaDev
Copy link
Copy Markdown
Contributor Author

I see a change in the MetaDataItem getTitle function so I guess this also requires a PR to MediasliderView?

yes: giejay/MediaSliderView#4

@giejay giejay merged commit 49bb974 into giejay:main Feb 1, 2026
@giejay
Copy link
Copy Markdown
Owner

giejay commented Feb 1, 2026

Thanks @HappyKoalaDev! I will install it on my local devices and draft up a release in the coming days.

@HappyKoalaDev
Copy link
Copy Markdown
Contributor Author

@giejay when can we expect the release?

@giejay
Copy link
Copy Markdown
Owner

giejay commented Mar 27, 2026

My apologies, I thought I released it already. It's now under review in the play store and should be available for the public in a few days.

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.

[FR] Language settings are required

2 participants