Skip to content

Multiple download#289

Merged
MikaeldeVerdier merged 4 commits intomasterfrom
multiple_download
Feb 18, 2026
Merged

Multiple download#289
MikaeldeVerdier merged 4 commits intomasterfrom
multiple_download

Conversation

@MikaeldeVerdier
Copy link
Contributor

Kanske för mycket för en pull request? Gör i alla fall så man kan markera och ladda ner flera bilder samt några localizations (både text och tid). Det mest märkbara är att login_ui är localized men det finns en knapp för att togglea locale. Har testat det mesta tror jag.

…ads and caches image_futures in album; some minor improvements
… a locale change button in login_ui; added time localization in various places (only for months); refactored single_news for both locales; some other minor improvements
Copy link
Contributor

@georgelgeback georgelgeback left a comment

Choose a reason for hiding this comment

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

LGTM other than one translation which needs fixing.

"resetEmail": "Email",
"resetSubmit": "Submit",
"resetValidate": "Please input an email",
"resetDone": "Your password has been reset. An email has been sent with your new password.",
Copy link
Contributor

Choose a reason for hiding this comment

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

On frontend web this is "If your email exists, a reset link has been sent.", which is more in line with what happens.

"resetEmail": "Email",
"resetSubmit": "Skicka",
"resetValidate": "Skriv en email",
"resetDone": "Ditt lösenord has återställts. Ett mail har skickats med ditt nya lösenord.",
Copy link
Contributor

Choose a reason for hiding this comment

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

Same thing here as in the english file

Copy link
Contributor

Choose a reason for hiding this comment

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

Pls regenerate these after so the old translations are not stuck

@MikaeldeVerdier
Copy link
Contributor Author

Fixed it! Also changed some other localizations to match the web counterpart

Copy link
Contributor

@georgelgeback georgelgeback left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

@MikaeldeVerdier MikaeldeVerdier merged commit 663bc27 into master Feb 18, 2026
@MikaeldeVerdier MikaeldeVerdier deleted the multiple_download branch February 18, 2026 15:11
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.

2 participants