Skip to content
This repository was archived by the owner on Aug 21, 2025. It is now read-only.

[Tooling] Add automation lane to fetch translations#105

Merged
iangmaia merged 5 commits intotrunkfrom
iangmaia/add-fetch-translations-lane
Aug 8, 2025
Merged

[Tooling] Add automation lane to fetch translations#105
iangmaia merged 5 commits intotrunkfrom
iangmaia/add-fetch-translations-lane

Conversation

@iangmaia
Copy link
Contributor

@iangmaia iangmaia commented Aug 7, 2025

Fixes AINFRA-1033

Description

This PR adds the lane download_translations that will fetch the translations from the Gravatar Android GlotPress projects (login-ui and home-ui) and add them to the project.

I've also ran the lane for the first time to initially add the files (though I noticed the only language with actual translations was Swedish).

Testing Steps

  1. Checkout this branch
  2. Add translations to one of the GlotPress projects mentioned above
  3. Run bundle exec download_translations skip_commit:true
  4. Verify that the added translation has been correctly downloaded to the project.

Copy link
Contributor

Choose a reason for hiding this comment

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

(Nothing to change here, just me ranting)

That reminds me that this file (generated by our gp_download_translations helper from release-toolkit) is quite obsolete now…

  • This was a file used by the WordPress Android project only, so that they can load it at runtime from their code to then have a custom language picker
  • But now WPAndroid have migrated to getting rid to the custom, in-app locale picker and using more modern Android APIs that offers system-level locale switching (similar to iOS supporting setting the locale per app in System Settings)… so I don't think it's used in WPAndroid anymore
  • And to my knowledge it's not used by any other project…

That shows the legacy of those helpers 😅

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ahh thanks for the context 👍 I was curious why this file was being generated in the first place; just checked a few projects that still have it, and imagined it might still be used elsewhere 😅

@AliSoftware
Copy link
Contributor

Ooof we might want to update the Android linter settings to ignore MissingTranslation rule 😅

@iangmaia iangmaia force-pushed the iangmaia/add-fetch-translations-lane branch from 5311958 to 35b3e67 Compare August 8, 2025 11:11
@iangmaia iangmaia force-pushed the iangmaia/add-fetch-translations-lane branch from 35b3e67 to 56b55ab Compare August 8, 2025 11:12
<string name="gravatar_tab_tap_for_options">Tryck för alternativ</string>
<string name="gravatar_tab_empty_avatars">Dina profilbilder kommer visas här</string>
<string name="gravatar_tab_previous_avatars">Tidigare profilbilder</string>
<string name="about_field_label_contact_email">E-post</string>

Check warning

Code scanning / Android Lint

Unused resources Warning

The resource R.string.about_app_dialog_privacy_policy appears to be unused
<string name="gravatar_tab_empty_avatars">Dina profilbilder kommer visas här</string>
<string name="gravatar_tab_previous_avatars">Tidigare profilbilder</string>
<string name="about_field_label_contact_email">E-post</string>
<string name="about_field_label_cell_phone">Telefon</string>

Check warning

Code scanning / Android Lint

Unused resources Warning

The resource R.string.about_field_label_cell_phone appears to be unused
@iangmaia iangmaia merged commit b623727 into trunk Aug 8, 2025
10 checks passed
@iangmaia iangmaia deleted the iangmaia/add-fetch-translations-lane branch August 8, 2025 12:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants