Translation make targets point at a directory that does not exist
Nobody has claimed this yet.
Assessment
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Newbie friendliness
- 65/100
- Issue type
- Bug
- Clarity
- Mostly clear
- Activity status
- Active
- Tech stack
- git, python
- Domain
- build-system, localization
Research direction
Start in backend-plugin-sample/Makefile and inspect the listed localization targets, including the atlas branch of pull_translations and install_transifex_client. Verify the package and locale paths under src/openedx_plugin_sample, then run the affected targets. Done means the targets no longer fail on missing directories or LICENSE.txt, with the issue's decision on exercising or removing the translation scaffolding addressed.
Written by the indexing model from the issue text.
Description
Problem
Every localization target in backend-plugin-sample/Makefile does cd sample_plugin, but there is no sample_plugin/ directory. The package lives at backend-plugin-sample/src/openedx_plugin_sample/. The path predates the renames in a9f4b8d (platform-plugin-sample -> backend-plugin-sample / openedx_plugin_sample).
So all of these fail immediately with cd: no such file or directory:
extract_translationscompile_translationsdetect_changed_source_translationsdummy_translationspull_translations(the atlas branch, which also referencessample_plugin/conf/locale)- and therefore
build_dummy_translationsandvalidate_translations
Nothing in CI runs these targets, which is why the breakage has gone unnoticed.
Also in scope
install_transifex_client ends with:
git checkout -- LICENSE README.md ## overwritten by Transifex installer
There is no LICENSE in that directory either -- the file is LICENSE.txt.
Suggested fix
Point the targets at src/openedx_plugin_sample, fix the atlas locale paths to match, and correct the LICENSE.txt filename. Worth considering whether validate_translations should run in CI so this cannot silently rot again.
Note that the plugin does not currently mark any strings for translation, so part of this work is deciding whether the i18n scaffolding should be exercised as a demonstration (this repo is meant to be an example of the plugin interfaces) or removed until there is something to translate.
- Dominant language
- Python
- Stars
- 1
- Forks
- 4
- Avg merge
- 3d 1h
- Merged PRs (30d)
- 8
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from openedx/sample-plugin
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
openedx/sample-plugin#46 · 1 reaction ·
-
Difficulty 3/5 1-2 days Newbie friendliness 56/100
openedx/sample-plugin#51 · 1 comment ·
-
Difficulty 3/5 1-2 days Newbie friendliness 55/100
openedx/sample-plugin#50 · 2 comments ·
-
Difficulty 4/5 3-5 days Newbie friendliness 32/100
openedx/sample-plugin#33 ·
-
openedx/sample-plugin#30 · 1 assignee ·
All issues in openedx/sample-plugin
Similar issues
-
Add: hunch Open
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
AbdelStark/awesome-typesafe#104 ·
-
enhancement
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
DiamondLightSource/dodal#2211 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
openml/openml-python#1749 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
sipyourdrink-ltd/bernstein#6191 ·