Skip to content

Conversation

@superuser-does
Copy link
Collaborator

@superuser-does superuser-does commented Jan 5, 2025

Capitalised the first three entries of the computer analysis results table in source strings instead of through CSS, per discussion in #16649.

In the absence of a convention, I changed nbFoo to numberFoo for the new strings. The key must be different, and I hope this is a suitable substitute.

  • I believe this will still build based on the CI (I have not built it myself).
  • This commit should not delete the old strings on Crowdin because they are in the XML file under translation/source. Crowdin is only concerned with the translation folder, which is why I think it's safe to remove the corresponding entries for the old keys in i18n.d.ts and key.scala.

Once pulled, this commit will enable me to run pre-translation on the Crowdin side for the new string.
At that point, I can make a separate commit to remove the old strings.

New keys introduced; the existing keys will be retained in /translation/source until Crowdin pre-translation is complete.
The old keys should be deleted only after the pre-translations have been pulled into master.
@ornicar ornicar merged commit 78e28df into master Jan 6, 2025
9 checks passed
@superuser-does superuser-does deleted the i18n-capital-advices branch January 20, 2025 08:16
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.

3 participants