avoid type error in initClassInfo; use KExternalLink to redirect in a…

…uthmessage w/ urls
learningequality · Oct 25, 2024 · c8a6c37 · c8a6c37
1 parent 826a1f1
commit c8a6c37
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 5 deletions.
diff --git a/kolibri/core/assets/src/views/AuthMessage.vue b/kolibri/core/assets/src/views/AuthMessage.vue
@@ -17,9 +17,9 @@
       />
     </p>
     <p v-else>
-      <KRouterLink
+      <KExternalLink
         :text="$tr('goBackToHomeAction')"
-        :to="{ path: '/' }"
+        :href="rootUrl"
         appearance="basic-link"
       />
     </p>
@@ -65,6 +65,9 @@
     },
     computed: {
       ...mapGetters(['isUserLoggedIn']),
+      rootUrl() {
+        return urls['kolibri:core:redirect_user']();
+      },
       detailsText() {
         return this.details || this.$tr(this.authorizedRole);
       },

diff --git a/kolibri/plugins/coach/assets/src/modules/pluginModule.js b/kolibri/plugins/coach/assets/src/modules/pluginModule.js
@@ -114,9 +114,11 @@ export default {
         return Promise.all([
           // Make sure we load any class list data, so that we know
           // whether this user has access to multiple classes or not.
-          store
-            .dispatch('classSummary/loadClassSummary', classId)
-            .then(summary => store.dispatch('setClassList', summary.facility_id)),
+          store.dispatch('classSummary/loadClassSummary', classId).then(summary => {
+            if (summary?.facility_id) {
+              store.dispatch('setClassList', summary?.facility_id);
+            }
+          }),
           store.dispatch('coachNotifications/fetchNotificationsForClass', classId),
         ]).catch(error => {
           store.dispatch('handleApiError', { error, reloadOnReconnect: true });