Merge pull request #11061 from LianaHarris360/classes_list_loading_state

Add loader to YourClasses component
learningequality · Aug 9, 2023 · 6e2035f · 6e2035f
2 parents 29208dc + a2abae3
commit 6e2035f
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/kolibri/plugins/learn/assets/src/views/YourClasses/index.vue b/kolibri/plugins/learn/assets/src/views/YourClasses/index.vue
@@ -39,10 +39,9 @@
       </CardLink>
     </CardGrid>
 
-    <p v-else>
+    <p v-else-if="!loading">
       {{ $tr('noClasses') }}
     </p>
-
   </section>
 
 </template>
@@ -80,6 +79,10 @@
         required: false,
         default: false,
       },
+      loading: {
+        type: Boolean,
+        default: null,
+      },
     },
     data() {
       return {

diff --git a/kolibri/plugins/learn/assets/src/views/classes/AllClassesPage.vue b/kolibri/plugins/learn/assets/src/views/classes/AllClassesPage.vue
@@ -7,6 +7,7 @@
     <YourClasses
       v-if="isUserLoggedIn"
       :classes="classrooms"
+      :loading="loading"
     />
     <AuthMessage v-else authorizedRole="learner" />
   </LearnAppBarPage>
@@ -42,6 +43,9 @@
     computed: {
       ...mapGetters(['isUserLoggedIn']),
       ...mapState('classes', ['classrooms']),
+      ...mapState({
+        loading: state => state.core.loading,
+      }),
       breadcrumbs() {
         return [
           {