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

BRouter translations #645

Merged
merged 1 commit into from Dec 9, 2023
Merged

BRouter translations #645

merged 1 commit into from Dec 9, 2023

Conversation

devemux86
Copy link
Contributor

@devemux86 devemux86 commented Nov 21, 2023

  • English, Arabic, Catalan, Dutch, French, German, Greek, Italian, Polish, Spanish
  • Move fixed strings to donottranslate-strings.xml
  • Remove unused strings
  • Enable app localization

There are several texts in the code that can be moved to strings.xml and translated as well.

The translations were contributed by Cruiser users:

@afischerdev
Copy link
Collaborator

This breaks on git action but runs on a local gradlew build:

BInstallerActivity.java:174: Error: Format string 'summary_segments' is not a valid format string

<string name="summary_segments" formatted="false">Size=%s\nFree=%s</string>

Any idea?

@devemux86
Copy link
Contributor Author

devemux86 commented Nov 21, 2023

Interesting that there is a Lint error.

However if we do not use the formatted="false" then the build of release apk shows the warning:
Multiple substitutions specified in non-positional format of string resource string/summary_segments. Did you mean to add the formatted="false" attribute?

We better try with positional arguments or split the string.

@devemux86
Copy link
Contributor Author

@afischerdev please run the workflow again.

Don't they run automatically after the first approval?

@afischerdev
Copy link
Collaborator

or split the string.

Yes, but this one has also warnings:

Do not concatenate text displayed with setText. Use resource string with placeholders.

Don't they run automatically after the first approval?

I've seen, but can't say why.

@devemux86
Copy link
Contributor Author

devemux86 commented Nov 21, 2023

This should not be considered as a serious problem,
but as also the current master branch shows warnings on production builds (please see above),
so something else may be needed, I will check it.

@devemux86
Copy link
Contributor Author

@afischerdev please let's try again the build.

@devemux86
Copy link
Contributor Author

@afischerdev is the Lint successful now on these changes?

@afischerdev
Copy link
Collaborator

@devemux86 fine now.

After merge I'll start the collection for the other hard coded strings.
I hope there are some candidates for remove.

@afischerdev afischerdev merged commit 7657985 into abrensch:master Dec 9, 2023
1 check passed
@afischerdev
Copy link
Collaborator

Thank you for contribution.

@devemux86 devemux86 deleted the translations branch December 9, 2023 14:45
@afischerdev afischerdev added this to the Version 1.7.4 milestone Jan 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants