feat: standardize make extract_translations (#212)

* Delete `locale` directory as it's not needed * Fix the translation command `make extract_translations` to use `django-admin` instead of `manage.py`. And fix `cd edx-ace` to the correct path `cd edx_ace` Refs: FC-0012 OEP-58
openedx · May 9, 2023 · db15604 · db15604
1 parent 1e293dd
commit db15604
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 76 deletions.
diff --git a/.gitignore b/.gitignore
@@ -71,3 +71,6 @@ edx_ace/tests/__init__.py
 default.db
 
 .hypothesis
+
+# Translation
+edx_ace/conf/locale/*/LC_MESSAGES/*
diff --git a/Makefile b/Makefile
@@ -88,14 +88,14 @@ write-pylintrc:
 
 extract_translations: ## extract strings to be translated, outputting .mo files
 	rm -rf docs/_build
-	cd edx-ace && ../manage.py makemessages -l en -v1 -d django
-	cd edx-ace && ../manage.py makemessages -l en -v1 -d djangojs
+	cd edx_ace && django-admin makemessages -l en -v1 -d django
+	cd edx_ace && django-admin makemessages -l en -v1 -d djangojs
 
 compile_translations: ## compile translation files, outputting .po files for each supported language
-	cd edx-ace && ../manage.py compilemessages
+	cd edx_ace && django-admin compilemessages
 
 detect_changed_source_translations:
-	cd edx-ace && i18n_tool changed
+	cd edx_ace && i18n_tool changed
 
 pull_translations: ## pull translations from Transifex
 	tx pull -t -a -f --mode reviewed

diff --git a/locale/config.yaml b/locale/config.yaml