openedx / frontend-app-discussions

A React-based micro frontend for the Open edX discussion forums.
GNU Affero General Public License v3.0
6 stars 61 forks source link

feat: remove Transifex calls for OEP-58 for discussion MFE #682

Closed sundasnoreen12 closed 3 months ago

sundasnoreen12 commented 3 months ago

INF-1279 Similar to 887

Description

Changes

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

Merge Checklist

Post-merge Checklist

codecov[bot] commented 3 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 92.82%. Comparing base (1a2068d) to head (cb6d336).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #682 +/- ## ======================================= Coverage 92.82% 92.82% ======================================= Files 158 158 Lines 3331 3331 Branches 919 919 ======================================= Hits 3092 3092 Misses 219 219 Partials 20 20 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.