From 4c53d58313b70911d40d561febd2c86bbc9f3cfa Mon Sep 17 00:00:00 2001 From: Cristhian Garcia Date: Tue, 22 Oct 2024 08:34:02 -0500 Subject: [PATCH] fix: add instructor role to operators too --- .../apps/superset/pythonpath/openedx_sso_security_manager.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tutoraspects/templates/aspects/apps/superset/pythonpath/openedx_sso_security_manager.py b/tutoraspects/templates/aspects/apps/superset/pythonpath/openedx_sso_security_manager.py index 5f0698517..75eb62b32 100644 --- a/tutoraspects/templates/aspects/apps/superset/pythonpath/openedx_sso_security_manager.py +++ b/tutoraspects/templates/aspects/apps/superset/pythonpath/openedx_sso_security_manager.py @@ -173,7 +173,7 @@ def _get_user_roles(self, username, locale): if decoded_access_token.get("superuser", False): return ["admin", f"admin-{locale}"] elif decoded_access_token.get("administrator", False): - return ["alpha", "operator", f"operator-{locale}"] + return ["alpha", "operator", f"operator-{locale}", "instructor", f"instructor-{locale}"] else: # User has to have staff access to one or more courses to view any content # here. Since this is only called on login, we take the opportunity