openedx / openedx/sample-plugin

Translation make targets point at a directory that does not exist

Offen
#65 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Vorherrschende Sprache
Python
Sterne
1
Forks
4
Ø Merge
3 T. 1 Std.
Gemergte PRs (30 T.)
8

Beschreibung

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_translations
  • compile_translations
  • detect_changed_source_translations
  • dummy_translations
  • pull_translations (the atlas branch, which also references sample_plugin/conf/locale)
  • and therefore build_dummy_translations and validate_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.

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne in backend-plugin-sample/Makefile und prüfe die aufgeführten Lokalisierungsziele, einschließlich des atlas-Zweigs von pull_translations und install_transifex_client. Überprüfe die Paket- und Locale-Pfade unter src/openedx_plugin_sample und führe dann die betroffenen Ziele aus. Erledigt ist die Aufgabe, wenn die Ziele nicht mehr wegen fehlender Verzeichnisse oder LICENSE.txt fehlschlagen und die Entscheidung des Issues darüber, ob das Übersetzungsgerüst ausgeführt oder entfernt werden soll, umgesetzt ist.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
git, python
Bereich
build-system, localization
Issue-Typ
Bug
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Aktiv
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
65/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.