fix: allow to translate dashboard headers

Author: Ian2012

Makefile

@@ -86,6 +86,10 @@ compile_translations:
 	TUTOR_ROOT=$(TUTOR_ROOT) tutor config save
 	python scripts/translate.py $(TUTOR_ROOT) compile
 
+list_translations:
+	TUTOR_ROOT=$(TUTOR_ROOT) tutor config save
+	python scripts/translate.py $(TUTOR_ROOT) list
+
 version: ## Print the current tutor version
 	@python -c 'import io, os; about = {}; exec(io.open(os.path.join("$(PACKAGE)", "__about__.py"), "rt", encoding="utf-8").read(), about); print(about["__version__"])'
 

scripts/translate.py

@@ -2,10 +2,9 @@
 import sys
 
 import click
-import yaml
-from transifex.native import init, tx
-from transifex.native.parsing import SourceString
-from utils import LANGUAGES, compile_translations, push_translations
+from transifex.native import init
+from utils import (LANGUAGES, compile_translations, get_text_for_translations,
+                   push_translations)
 
 init(
     os.getenv("TRANSIFEX_TOKEN"),
@@ -23,6 +22,8 @@ def command(root, action):
         push_translations(root)
     elif action == "compile":
         compile_translations(root)
+    elif action == "list":
+        get_text_for_translations(root)
     else:
         print("Unknown action: {}".format(action))
         sys.exit(1)

scripts/utils.py

@@ -55,7 +55,7 @@ def extract_text(asset, type):
 
                 meta = element.get("meta", {})
 
-                if element.get("type") == "TAB" and meta.get("text"):
+                if meta.get("text"):
                     strings.append(meta["text"])
 
         elif type == "charts":

tutoraspects/templates/aspects/apps/superset/pythonpath/create_assets.py

@@ -1,22 +1,23 @@
 """Import a list of assets from a yaml file and create them in the superset assets folder."""
 import os
 import uuid
-import yaml
 from zipfile import ZipFile
+
+import yaml
 from superset.app import create_app
 
 app = create_app()
 app.app_context().push()
 
+from copy import deepcopy
+
 from superset import security_manager
 from superset.commands.importers.v1.assets import ImportAssetsCommand
 from superset.commands.importers.v1.utils import get_contents_from_bundle
 from superset.extensions import db
 from superset.models.dashboard import Dashboard
 from superset.utils.database import get_or_create_db
 
-from copy import deepcopy
-
 BASE_DIR = "/app/assets/superset"
 
 ASSET_FOLDER_MAPPING = {
@@ -37,7 +38,9 @@
 ASSETS_FILE_PATH = "/app/pythonpath/assets.yaml"
 ASSETS_ZIP_PATH = "/app/assets/assets.zip"
 
-ASSETS_TRANSLATIONS = yaml.load(open(TRANSLATIONS_FILE_PATH, "r"), Loader=yaml.FullLoader)
+ASSETS_TRANSLATIONS = yaml.load(
+    open(TRANSLATIONS_FILE_PATH, "r"), Loader=yaml.FullLoader
+)
 
 
 def main():
@@ -99,9 +102,7 @@ def write_asset_to_file(asset, asset_name, folder, file_name, roles):
     #           access the original dashboards.
     dashboard_roles = asset.pop("_roles", None)
     if dashboard_roles:
-        roles[asset["uuid"]] = [
-            security_manager.find_role("Admin")
-        ]
+        roles[asset["uuid"]] = [security_manager.find_role("Admin")]
 
     path = f"{BASE_DIR}/{folder}/{file_name}.yaml"
     with open(path, "w") as file:
@@ -136,6 +137,8 @@ def generate_translated_dashboard_elements(copy, language):
     """Generate translated elements for a dashboard"""
     position = copy.get("position", {})
 
+    SUPPORTED_TYPES = ["TAB", "HEADER"]
+
     for element in position.values():
         if not isinstance(element, dict):
             continue
@@ -155,12 +158,12 @@ def generate_translated_dashboard_elements(copy, language):
             meta["sliceName"] = translation
             meta["uuid"] = element_id
 
-        elif element.get("type") == "TAB":
+        elif element.get("type") in SUPPORTED_TYPES:
             chart_body_id = element.get("id")
             if not meta or not meta.get("text"):
                 continue
 
-            element_type = "Tab"
+            element_type = element.get("type")
             element_id = chart_body_id
             translation = get_translation(meta["text"], language)
 
@@ -220,7 +223,6 @@ def update_dashboard_roles(roles):
         db.session.commit()
 
 
-
 def get_translation(text, language):
     """Get a translation for a text in a language"""
     LANGUAGE = ASSETS_TRANSLATIONS.get(language, {})

tutoraspects/templates/aspects/apps/superset/pythonpath/locale.yaml

@@ -17,6 +17,8 @@ de:
   Operator Dashboard: ''
   Organizations: ''
   Problem engagement: ''
+  Problem engagement overview: ''
+  Problem performance: ''
   Problem submission counts: ''
   Response distribution: ''
   Synchronized Courses: ''
@@ -49,6 +51,8 @@ es:
   Operator Dashboard: Panel de Operador
   Organizations: Organizaciones
   Problem engagement: "Participaci\xF3n en problemas"
+  Problem engagement overview: ''
+  Problem performance: ''
   Problem submission counts: "Recuento de env\xEDo de problemas"
   Response distribution: "Distribuci\xF3n de respuestas"
   Synchronized Courses: Cursos sincronizados
@@ -81,6 +85,8 @@ fr:
   Operator Dashboard: ''
   Organizations: ''
   Problem engagement: ''
+  Problem engagement overview: ''
+  Problem performance: ''
   Problem submission counts: ''
   Response distribution: ''
   Synchronized Courses: ''
@@ -112,6 +118,8 @@ it:
   Operator Dashboard: ''
   Organizations: ''
   Problem engagement: ''
+  Problem engagement overview: ''
+  Problem performance: ''
   Problem submission counts: ''
   Response distribution: ''
   Synchronized Courses: ''
@@ -143,6 +151,8 @@ ja:
   Operator Dashboard: ''
   Organizations: ''
   Problem engagement: ''
+  Problem engagement overview: ''
+  Problem performance: ''
   Problem submission counts: ''
   Response distribution: ''
   Synchronized Courses: ''
@@ -174,6 +184,8 @@ ko:
   Operator Dashboard: ''
   Organizations: ''
   Problem engagement: ''
+  Problem engagement overview: ''
+  Problem performance: ''
   Problem submission counts: ''
   Response distribution: ''
   Synchronized Courses: ''
@@ -205,6 +217,8 @@ nl:
   Operator Dashboard: ''
   Organizations: ''
   Problem engagement: ''
+  Problem engagement overview: ''
+  Problem performance: ''
   Problem submission counts: ''
   Response distribution: ''
   Synchronized Courses: ''
@@ -236,6 +250,8 @@ pt:
   Operator Dashboard: ''
   Organizations: ''
   Problem engagement: ''
+  Problem engagement overview: ''
+  Problem performance: ''
   Problem submission counts: ''
   Response distribution: ''
   Synchronized Courses: ''
@@ -267,6 +283,8 @@ ru:
   Operator Dashboard: ''
   Organizations: ''
   Problem engagement: ''
+  Problem engagement overview: ''
+  Problem performance: ''
   Problem submission counts: ''
   Response distribution: ''
   Synchronized Courses: ''
@@ -298,6 +316,8 @@ sk:
   Operator Dashboard: ''
   Organizations: ''
   Problem engagement: ''
+  Problem engagement overview: ''
+  Problem performance: ''
   Problem submission counts: ''
   Response distribution: ''
   Synchronized Courses: ''
@@ -329,6 +349,8 @@ sl:
   Operator Dashboard: ''
   Organizations: ''
   Problem engagement: ''
+  Problem engagement overview: ''
+  Problem performance: ''
   Problem submission counts: ''
   Response distribution: ''
   Synchronized Courses: ''
@@ -360,6 +382,8 @@ zh:
   Operator Dashboard: ''
   Organizations: ''
   Problem engagement: ''
+  Problem engagement overview: ''
+  Problem performance: ''
   Problem submission counts: ''
   Response distribution: ''
   Synchronized Courses: ''