feat: make extract_translations to use i18n_tool (#613)

Refs: FC-0012 OEP-58
openedx · Oct 18, 2023 · 386dbe2 · 386dbe2
1 parent e7dd8c8
commit 386dbe2
Show file tree

Hide file tree

Showing 9 changed files with 9 additions and 6 deletions.
diff --git a/Makefile b/Makefile
@@ -6,6 +6,8 @@
 
 NODE_BIN=$(CURDIR)/node_modules/.bin
 
+THEME_NAME := edx_credentials_themes
+
 # Generates a help message. Borrowed from https://github.com/pydanny/cookiecutter-djangopackage.
 help: ## display this help message
 	@echo "Please use \`make <target>\` where <target> is one of"
@@ -18,16 +20,16 @@ build.watch:
 	$(NODE_BIN)/webpack --config webpack.config.js --progress --watch
 
 compile_translations:
-	cd edx_credentials_themes && django-admin.py compilemessages
+	cd $(THEME_NAME) && i18n_tool generate
 
 detect_changed_source_translations:
-	cd edx_credentials_themes && i18n_tool changed
+	cd $(THEME_NAME) && i18n_tool changed
 
 dummy_translations:
-	cd edx_credentials_themes && i18n_tool dummy
+	cd $(THEME_NAME) && i18n_tool dummy
 
-extract_translations:
-	cd edx_credentials_themes && django-admin.py makemessages -l en -d django
+extract_translations: ## extract strings to be translated, outputting .po files
+	cd $(THEME_NAME) && i18n_tool extract --no-segment
 
 generate_translations: extract_translations dummy_translations compile_translations
 
@@ -47,7 +49,7 @@ requirements: base_requirements
 
 test:
 	# Confirm compiled assets have not changed, indicating SASS matches CSS.
-	git diff --exit-code edx_credentials_themes/
+	git diff --exit-code $(THEME_NAME)/ ":(exclude)$(THEME_NAME)/conf"
 
 upgrade: export CUSTOM_COMPILE_COMMAND=make upgrade
 upgrade: ## update the requirements/*.txt files with the latest packages satisfying requirements/*.in

diff --git a/edx_credentials_themes/locale/config.yaml → ...redentials_themes/conf/locale/config.yaml b/edx_credentials_themes/locale/config.yaml → ...redentials_themes/conf/locale/config.yaml
diff --git a/...ls_themes/locale/en/LC_MESSAGES/django.mo → ...emes/conf/locale/en/LC_MESSAGES/django.mo b/...ls_themes/locale/en/LC_MESSAGES/django.mo → ...emes/conf/locale/en/LC_MESSAGES/django.mo
diff --git a/...ls_themes/locale/en/LC_MESSAGES/django.po → ...emes/conf/locale/en/LC_MESSAGES/django.po b/...ls_themes/locale/en/LC_MESSAGES/django.po → ...emes/conf/locale/en/LC_MESSAGES/django.po
diff --git a/...ls_themes/locale/eo/LC_MESSAGES/django.mo → ...emes/conf/locale/eo/LC_MESSAGES/django.mo b/...ls_themes/locale/eo/LC_MESSAGES/django.mo → ...emes/conf/locale/eo/LC_MESSAGES/django.mo
diff --git a/...ls_themes/locale/eo/LC_MESSAGES/django.po → ...emes/conf/locale/eo/LC_MESSAGES/django.po b/...ls_themes/locale/eo/LC_MESSAGES/django.po → ...emes/conf/locale/eo/LC_MESSAGES/django.po
diff --git a/...hemes/locale/es_419/LC_MESSAGES/django.mo → .../conf/locale/es_419/LC_MESSAGES/django.mo b/...hemes/locale/es_419/LC_MESSAGES/django.mo → .../conf/locale/es_419/LC_MESSAGES/django.mo
diff --git a/...hemes/locale/es_419/LC_MESSAGES/django.po → .../conf/locale/es_419/LC_MESSAGES/django.po b/...hemes/locale/es_419/LC_MESSAGES/django.po → .../conf/locale/es_419/LC_MESSAGES/django.po
diff --git a/edx_credentials_themes/locale b/edx_credentials_themes/locale
@@ -0,0 +1 @@
+conf/locale