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

Fix i18n issue #1216

Merged
merged 2 commits into from Nov 8, 2021
Merged

Fix i18n issue #1216

merged 2 commits into from Nov 8, 2021

Conversation

ZitaNemeckova
Copy link
Member

@ZitaNemeckova ZitaNemeckova commented Nov 8, 2021

Fixes AAH-982

Adding missing compiled files to fix "string {0} string" issue in prod.

Fixed a test that was failing because of it to prove it's fixed.
Before the fix:
Screenshot 2021-11-08 at 12 59 40

After:
🟢

@himdel Someone's gonna have the most new lines Milan Zazrivec style :D

@ZitaNemeckova ZitaNemeckova added backport-4.4 This PR should be backported to stable-4.4 (2.1) bug labels Nov 8, 2021
@ZitaNemeckova ZitaNemeckova changed the title Fix l10n interpolation issue Fix i18n interpolation issue Nov 8, 2021
@ZitaNemeckova ZitaNemeckova changed the title Fix i18n interpolation issue Fix i18n issue Nov 8, 2021
locale/fr.po Outdated Show resolved Hide resolved
Extract and compile new translatable strings to prevent wrong display in prod that looks like "string{0}string"
Copy link
Collaborator

@himdel himdel left a comment

Choose a reason for hiding this comment

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

LGTM, not seeing any issues :)

@ZitaNemeckova ZitaNemeckova merged commit 8b34c28 into ansible:master Nov 8, 2021
@ZitaNemeckova ZitaNemeckova deleted the fix_l10n branch November 8, 2021 15:40
@patchback
Copy link

patchback bot commented Nov 8, 2021

Backport to stable-4.4: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-4.4/8b34c28ca9ffdf3b75fcc53029e902e41a26d26b/pr-1216

Backported as #1218

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

patchback bot pushed a commit that referenced this pull request Nov 8, 2021
* Fixing the test to prove translatedable strings are correct in prod

* Fixes https://issues.redhat.com/browse/AAH-982

Extract and compile new translatable strings to prevent wrong display in prod that looks like "string{0}string"

(cherry picked from commit 8b34c28)
himdel pushed a commit that referenced this pull request Nov 8, 2021
* Fixing the test to prove translatedable strings are correct in prod

* Fixes https://issues.redhat.com/browse/AAH-982

Extract and compile new translatable strings to prevent wrong display in prod that looks like "string{0}string"

(cherry picked from commit 8b34c28)

Co-authored-by: ZitaNemeckova <znemecko@redhat.com>
@github-actions github-actions bot added the backported-4.4 This PR has been backported to stable-4.4 (2.1) label Nov 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-4.4 This PR should be backported to stable-4.4 (2.1) backported-4.4 This PR has been backported to stable-4.4 (2.1)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants