learningequality / kolibri

Kolibri Learning Platform: the offline app for universal education
https://learningequality.org/kolibri/
MIT License
784 stars 650 forks source link

nutritionfacts_i18n script doesn't output English strings #5782

Closed indirectlylit closed 4 years ago

indirectlylit commented 5 years ago

Observed behavior

currently outputs something like

{
  "ar": {
    "link_text": "تعلم المزيد وقم بتحميله هنا",
    "msg": "هناك إصدار جديد متاح من كوليبري.",
    "title": "التحديث للنسخة الجديدة أصبح متاحاً"
  },
  "bg-bg": {
    "link_text": "Научи повече и изтегли оттук",
    "msg": "Налична е нова версия на Колибри.",
    "title": "Има налични подобрения"
  },
  "bn-bd": {
    "link_text": "আরও জানুন এবং সেটি এখানে ডাউনলোড করুন",
    "msg": "কলিব্রির একটি নতুন সংস্করণ পাওয়া যাচ্ছে।",
    "title": "আপগ্রেড উপলব্ধ"
  },
  "en": {},
  "es-es": {
    "link_text": "Descubre más y descarga aquí",
    "msg": "Una nueva versión de Kolibri está disponible.",
    "title": "Actualización disponible"
  },
  //...
}

Expected behavior

should output something like

{
  "ar": {
    "link_text": "تعلم المزيد وقم بتحميله هنا",
    "msg": "هناك إصدار جديد متاح من كوليبري.",
    "title": "التحديث للنسخة الجديدة أصبح متاحاً"
  },
  "bg-bg": {
    "link_text": "Научи повече и изтегли оттук",
    "msg": "Налична е нова версия на Колибри.",
    "title": "Има налични подобрения"
  },
  "bn-bd": {
    "link_text": "আরও জানুন এবং সেটি এখানে ডাউনলোড করুন",
    "msg": "কলিব্রির একটি নতুন সংস্করণ পাওয়া যাচ্ছে।",
    "title": "আপগ্রেড উপলব্ধ"
  },
  "en": {
    "link_text": "Learn more and download it here",
    "msg": "A new version of Kolibri is available.",
    "title": "Upgrade available"
  },
  "es-es": {
    "link_text": "Descubre más y descarga aquí",
    "msg": "Una nueva versión de Kolibri está disponible.",
    "title": "Actualización disponible"
  },
  //...
}

User-facing consequences

devs have to do manual work copying over old strings

Steps to reproduce

run:

python build_tools/i18n/nutritionfacts_i18n.py

Context

0.12.5

lyw07 commented 4 years ago

sorry I'm not sure if I understand this issue correctly... Is this because we don't have the json file for English on Crowdin? Thank you!