Page Comparison

...

• Testing on devstack palm release. With XBlock xblock-drag-and-drop-v2

• Applying code on open-release/palm.master.

  • Because master branch does work correctly with devstack palm

  • https://github.com/

  openedx

  • Zeit-Labs/edx-platform/

  compare/open-release/palm.master...

  • pull/2

  • https://github.com/Zeit-Labs

  :edx-platform:shadinaif/palm-xblock-translations-poc?expand=1

  • Because master branch does work correctly with devstack palm

  • /xblock-drag-and-drop-v2/pull/2

• Commenting out xblock-drag-and-drop-v2==3.1.2 in development.txt requirement file. Because paver update-assets will reinstall the original version within its process if we don’t

...

• ✅ i18n service updated

• ❌ Waffel activated

• ❌ Translations pulled

• ❌ XBLOCK_TRANSLATIONS_DIRECTORY correctly set

• ❌ XBlock OEP-19 58 support added

• ❌ XBlock local translations removed

...

• ✅ i18n service updated

• ✅ Waffel activated

• Translations pulled

• XBLOCK_TRANSLATIONS_DIRECTORY correctly set

• XBlock OEP-19 58 support added

• XBlock local translations removed

...

• ✅ i18n service updated

• ✅ Waffel activated

• Translations pulled

• XBLOCK_TRANSLATIONS_DIRECTORY correctly set

• ✅ XBlock OEP-19 58 support added

• XBlock local translations removed

...

• ✅ i18n service updated

• ✅ Waffel activated

• ✅ Translations pulled

• XBLOCK_TRANSLATIONS_DIRECTORY correctly set

• ✅ XBlock OEP-19 58 support added

• XBlock local translations removed

...

Running cd conf && i18n_tool generate , compiled successfully. But before compilation, I changed few translations of Arabic language so I can see if translations are pulled from local or oepOEP-19 58 directory without setting annoying logs

...

• ✅ i18n service updated

• ✅ Waffel activated

• ✅ Translations pulled

• ✅ XBLOCK_TRANSLATIONS_DIRECTORY correctly set

• ✅ XBlock OEP-19 58 support added

• XBlock local translations removed

...

Java Script translations are not loaded from Atlas yet! The text.js does not exist yet

Pending Known Issues

Module Name

Module name that we calculate with

xblock_module_name = xblock_resource.__name__

raises an error because xblock_resource is a string. Using xblock_resource itself does not return the desired name that is used by Atlas when pulling translations. We need to unify the name between the two mechanisms

Domain

The domain used in the XBlock is still text (and it should still be that in my opinion). But the translation is being pulled as django

Python Compilation

Atlas command should compile translations after pulling them

cd conf && i18n_tool generate

JS translation not being compiled

This mechanism is not implemented yet