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

Localisation fix #3916

Merged
merged 2 commits into from
Feb 4, 2025
Merged

Localisation fix #3916

merged 2 commits into from
Feb 4, 2025

Conversation

federicocappelli
Copy link
Member

@federicocappelli federicocappelli commented Feb 3, 2025

Task/Issue URL: https://app.asana.com/0/414235014887631/1209235565517198

Description:

Something weird is happening with our localisation process, here I:

  • exported a new xliff
  • downloaded the translations immediately
  • imported the translations

In theory, we should see 0 changes, instead we have a lot.

UPDATE:
Turns out that after applying the localisation the project needs to be built for re-generating the correct files. I think that step is missing from the localisation guidelines.

Definition of Done (Internal Only):


Internal references:

Software Engineering Expectations
Technical Design Template

Copy link

github-actions bot commented Feb 3, 2025

Warnings
⚠️ PR has more than 500 lines of code changing. Consider splitting into smaller PRs if possible.

Generated by 🚫 dangerJS against 61235e3

Copy link
Contributor

@alessandroboron alessandroboron left a comment

Choose a reason for hiding this comment

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

It seems translations work fine so approving it.

@federicocappelli federicocappelli merged commit 48d9366 into main Feb 4, 2025
18 checks passed
@federicocappelli federicocappelli deleted the fcappelli/localisation_fix branch February 4, 2025 02:38
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