feat: add Transifex config for freetextresponse translations#45
Conversation
Wires freetextresponse into the openedx-translations pipeline by adding the .tx/config resource definition (xblocks-extra-freetextresponse). Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #45 +/- ##
=======================================
Coverage 82.24% 82.24%
=======================================
Files 49 49
Lines 1425 1425
Branches 110 110
=======================================
Hits 1172 1172
Misses 221 221
Partials 32 32
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
Thanks for the pull request, @irfanuddinahmad! This repository is currently maintained by Once you've gone through the following steps feel free to tag them in a comment and let them know that your changes are ready for engineering review. 🔘 Get product approvalIf you haven't already, check this list to see if your contribution needs to go through the product review process.
🔘 Provide contextTo help your reviewers and other members of the community understand the purpose and larger context of your changes, feel free to add as much of the following information to the PR description as you can:
🔘 Get a green buildIf one or more checks are failing, continue working on your changes until this is no longer the case and your build turns green. DetailsWhere can I find more information?If you'd like to get more details on all aspects of the review process for open source pull requests (OSPRs), check out the following resources: When can I expect my changes to be merged?Our goal is to get community contributions seen and reviewed as efficiently as possible. However, the amount of time that it takes to review and merge a PR can vary significantly based on factors such as:
💡 As a result it may take up to several weeks or months to complete a review and merge your PR. |
|
Hi @openedx/axim-engineering - this is ready for review. Thanks! |
Adds the freetextresponse entry (migrated into xblocks-extra in openedx/xblocks-extra#45) so strings are extracted and translations are published back via the openedx-translations pipeline. Co-authored-by: Irfan Ahmad <irfan.ahmad@A006-01919.local> Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
freetextresponse has been migrated into xblocks-extra (openedx#45). This removes the legacy standalone package from the translations pipeline to eliminate the atlas glob-ambiguity bug where both paths matching the same module causes atlas pull to fail. - Remove transifex.yml filter entry for xblock-free-text-response - Remove xblock-free-text-response from allPythonRepos in extract workflow - Remove translations/xblock-free-text-response/ directory (37 locales) Depends on: openedx/xblocks-extra#45 (adds xblocks-extra/freetextresponse entry) Closes openedx/xblocks-extra#43 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
* feat: remove legacy xblock-free-text-response translations freetextresponse has been migrated into xblocks-extra (#45). This removes the legacy standalone package from the translations pipeline to eliminate the atlas glob-ambiguity bug where both paths matching the same module causes atlas pull to fail. - Remove transifex.yml filter entry for xblock-free-text-response - Remove xblock-free-text-response from allPythonRepos in extract workflow - Remove translations/xblock-free-text-response/ directory (37 locales) Depends on: openedx/xblocks-extra#45 (adds xblocks-extra/freetextresponse entry) Closes openedx/xblocks-extra#43 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * fix: preserve ka and sq translations in xblocks-extra path Copies Georgian (ka) and Albanian (sq) locale files from the legacy xblock-free-text-response path to the new xblocks-extra path, ensuring no translation coverage is lost when the legacy path is removed. These locales are not yet published by Transifex for the new resource (onlyreviewed mode); copying them directly preserves coverage until translators re-approve them on the new Transifex resource. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> --------- Co-authored-by: Irfan Ahmad <irfan.ahmad@A006-01919.local> Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
Summary
src/freetextresponse/.tx/configto wirefreetextresponseinto the openedx-translations pipelinexblocks-extra-freetextresponse, matching the pattern used by all other xblocks in this repo (e.g.xblocks-extra-sql-grader,xblocks-extra-submit-and-compare)conf/locale/config.yamlalready exists, and the Makefile auto-discoversfreetextresponseviafind src/ -mindepth 2 -maxdepth 2 -type d -name 'conf'Closes #41 (together with a companion PR in openedx/openedx-translations adding the
transifex.ymlentry)Merge sequence
translations/xblocks-extra/freetextresponse/in openedx-translationstransifex.ymlentry forxblocks-extra/freetextresponsexblock-free-text-responsecoverage)xblock-free-text-responseentries (resolves Remove legacy standalone xblock-free-text-response translations from openedx-translations #43)make pull_xblock_translationsinlms-shelland confirm translated UI renders correctlyTest plan
extract-translation-source-filesworkflow validates the entry🤖 Generated with Claude Code