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

Missing string in 3.7.1 donation button #1086

Open
Popolechien opened this issue Jan 21, 2025 · 6 comments · May be fixed by #1087
Open

Missing string in 3.7.1 donation button #1086

Popolechien opened this issue Jan 21, 2025 · 6 comments · May be fixed by #1087
Assignees
Milestone

Comments

@Popolechien
Copy link
Member

A user reported:

The iOS app is missing a string for the newly introduced donate button. My device is in German.
version 3.7.1 Build 204. That’s where the issue appears when donating a custom amount.

See image:

Image

@kelson42
Copy link
Contributor

@BPerlakiH This is urgent and requires s patch release ASAP. We also need to understsnd how this happened exactly.

@kelson42 kelson42 added this to the 3.8.0 milestone Jan 21, 2025
@BPerlakiH
Copy link
Collaborator

@kelson42 I've created a PR for this.
It seems, I forgot to add the .localised part to the usage of this string.
I double checked the other-ones for donation, they are OK in this regard.

@kelson42
Copy link
Contributor

@BPerlakiH Do you now a way to lint this? We should have detected such an error in the CI.

@BPerlakiH
Copy link
Collaborator

BPerlakiH commented Jan 21, 2025

As for why it happened:
If the string is found by the pure key in the current locale (eg. English), the value for that will be picked up automatically.
If the translation is missing, it will fall back to the key value itself.
What we do, is use this custom extension .localized, that will do the fallback for us to English, if there is no value for the current local (as we don't yet have one for German).

But if we miss to add it, we have this kind of issue, and not just for German, but for all languages, that do not yet have this term.

@BPerlakiH
Copy link
Collaborator

I will have a look, if we can automate this check somehow.

@BPerlakiH
Copy link
Collaborator

Created a new task for this: #1088
I have a couple of ideas for this, I need to do some experiments to see which is more feasible.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants