feat!: remove Transifex calls for OEP-58 | FC-0012

[hopewise]

Breaking change!

This change breaks the Jenkins transifex integration which has been deprecated in favor of the new GitHub Transifex App integration as part of OEP-58.

Changes

• Removes all direct use of tx pull and tx push commands from the micro-frontend in favor
  of the atlas pull command.
• Remove source and language translations from the repositories, hence no .json files will be committed into the repos.
• src/i18n/index.js should export and empty array so the make pull_translations override it with the dynamic list of languages.
• Remove the experimental OPENEDX_ATLAS_PULL flag to make atlas pull the default.
• Remove all Transifex related Makefile targets and other files.

Test results

• Verify that make pull_translations works as expected.

make pull_translations test results

# I've run the following commands:
$ make requirements
$ make pull_translations
$ git diff

# Output of the commmands:

make pull_translations
rm -rf src/i18n/messages
mkdir src/i18n/messages
cd src/i18n/messages \
      && atlas pull \
               translations/paragon/src/i18n/messages:paragon \
               translations/frontend-component-footer/src/i18n/messages:frontend-component-footer \
               translations/frontend-app-learner-dashboard/src/i18n/messages:frontend-app-learner-dashboard
Pulling translation files
 - directory: translations/paragon/src/i18n/messages:paragon translations/frontend-component-footer/src/i18n/messages:frontend-component-footer translations/frontend-app-learner-dashboard/src/i18n/messages:frontend-app-learner-dashboard
 - repository: openedx/openedx-translations
 - branch: main
 - filter: Not Specified
 - expand-glob: Not Specified
Creating a temporary Git repository to pull translations into "./translations_TEMP"...
Done.
Setting git sparse-checkout rules...
Done.
Pulling translation files from the repository...
remote: Enumerating objects: 34, done.
remote: Counting objects: 100% (34/34), done.
remote: Compressing objects: 100% (14/14), done.
remote: Total 34 (delta 24), reused 27 (delta 20), pack-reused 0
Receiving objects: 100% (34/34), 31.24 KiB | 515.00 KiB/s, done.
Resolving deltas: 100% (24/24), done.
Your branch is up to date with 'origin/main'.
Done.
Copying translations from "./translations_TEMP/translations/paragon/src/i18n/messages" to "./paragon"...
Done.
Copying translations from "./translations_TEMP/translations/frontend-component-footer/src/i18n/messages" to "./frontend-component-footer"...
Done.
Copying translations from "./translations_TEMP/translations/frontend-app-learner-dashboard/src/i18n/messages" to "./frontend-app-learner-dashboard"...
Done.
Removing temporary directory...
Done.

Translations pulled successfully!
./node_modules/.bin/intl-imports.js paragon frontend-component-footer frontend-app-learner-dashboard

Merge timeline

This should only be merged after Translation Infrastructure update OEP-58 is fully implemented.

The timing announcement will be shared by @brian-smith-tcril on #translations-project-fc-0012 Open edX Slack channel.

Keep this pull request as a draft to prevent accidental merge.

Pre-merge checklist

• Wait for approval on Deprecate pre-OEP-58 Transifex scripts from all repositories wg-translations#20

References

This contribution is part of the FC-0012 project which is sparked by the Translation Infrastructure update OEP-58.

Up-to-date project overview and details are available in the Approach Memo and Technical Discovery: Translations Infrastructure Implementation document.

Join the conversation on Open edX Slack #translations-project-fc-0012.

Check the links above for full information about the overall project.

[openedx-webhooks]

Thanks for the pull request, @hopewise! Please note that it may take us up to several weeks or months to complete a review and merge your PR.

Feel free to add as much of the following information to the ticket as you can:

• supporting documentation
• Open edX discussion forum threads
• timeline information ("this must be merged by XX date", and why that is)
• partner information ("this is a course on edx.org")
• any other information that can help Product understand the context for the PR

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.

Please let us know once your PR is ready for our review and all tests are green.

[itsjeyd]

@hopewise Thank you for this contribution! Please let us know when the changes are ready for review.

[e0d]

@hopewise looks like there are some check failures.

[OmarIthawi]

@itsjeyd @hopewise please change status from "Draft" to "Ready for merge"

This pull request is ready for merge, but pending approval from 2U (@timmc-edx)

cc: @brian-smith-tcril

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.69%. Comparing base (5083645) to head (d312ce2).

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #227      +/-   ##
==========================================
- Coverage   96.69%   96.69%   -0.01%     
==========================================
  Files         183      182       -1     
  Lines        1725     1724       -1     
  Branches      308      308              
==========================================
- Hits         1668     1667       -1     
  Misses         53       53              
  Partials        4        4              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[jsnwesson]

Hey all, I'm commenting on behalf of the Aperture team (2U), thanks for getting this ready for review! I'll be bringing this up with my team and we should have a response within the week.

[deborahgu]

within the day, hopefully. 🤞

[openedx-webhooks]

@hopewise 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future.