From fb0517caa0469e920c61b30b59b5a43fbbcf2751 Mon Sep 17 00:00:00 2001
From: Jannat Patel <pateljannat2308@gmail.com>
Date: Mon, 23 Feb 2026 14:54:54 +0530
Subject: [PATCH] fix: show only instructor tab for admins on home page

---
 frontend/src/pages/Home/Home.vue              | 41 ++++++++-----------
 .../course_evaluator/course_evaluator.json    | 14 ++++++-
 .../lms_certificate_request.json              | 14 ++++++-
 3 files changed, 44 insertions(+), 25 deletions(-)

diff --git a/frontend/src/pages/Home/Home.vue b/frontend/src/pages/Home/Home.vue
index 8bbe7e41..ee742be5 100644
--- a/frontend/src/pages/Home/Home.vue
+++ b/frontend/src/pages/Home/Home.vue
@@ -1,9 +1,4 @@
 <template>
-	<!-- <header
-		class="sticky flex items-center justify-between top-0 z-10 border-b bg-surface-white px-3 py-2.5 sm:px-5"
-	>
-		<Breadcrumbs :items="[{ label: __('Home'), route: { name: 'Home' } }]" />
-	</header> -->
 	<div class="w-full px-5 pt-5 pb-10">
 		<div class="space-y-2">
 			<div class="flex items-center justify-between">
@@ -11,9 +6,8 @@
 					{{ __('Hey') }}, {{ user.data?.full_name }} 👋
 				</div>
 				<div>
-					<TabButtons v-if="isAdmin" v-model="currentTab" :buttons="tabs" />
 					<div
-						v-else
+						v-if="!isAdmin"
 						@click="showStreakModal = true"
 						class="bg-surface-amber-2 px-2 py-1 rounded-md cursor-pointer"
 					>
@@ -35,19 +29,16 @@
 			:liveClasses="adminLiveClasses"
 			:evals="adminEvals"
 		/>
-		<StudentHome v-else :myLiveClasses="myLiveClasses" />
+		<StudentHome
+			v-else-if="currentTab === 'student'"
+			:myLiveClasses="myLiveClasses"
+		/>
 	</div>
 	<Streak v-model="showStreakModal" :streakInfo="streakInfo" />
 </template>
 <script setup lang="ts">
 import { computed, inject, onMounted, ref } from 'vue'
-import {
-	Breadcrumbs,
-	call,
-	createResource,
-	TabButtons,
-	usePageMeta,
-} from 'frappe-ui'
+import { call, createResource, usePageMeta } from 'frappe-ui'
 import { sessionStore } from '@/stores/session'
 import StudentHome from '@/pages/Home/StudentHome.vue'
 import AdminHome from '@/pages/Home/AdminHome.vue'
@@ -56,10 +47,10 @@ import Streak from '@/pages/Home/Streak.vue'
 const user = inject<any>('$user')
 const { brand } = sessionStore()
 const evalCount = ref(0)
-const currentTab = ref<'student' | 'instructor'>('instructor')
+const currentTab = ref<'student' | 'instructor'>('student')
 const showStreakModal = ref(false)
 
-onMounted(() => {
+const fetchEvalCount = () => {
 	call('frappe.client.get_count', {
 		doctype: 'LMS Certificate Request',
 		filters: {
@@ -70,7 +61,7 @@ onMounted(() => {
 	}).then((data: any) => {
 		evalCount.value = data
 	})
-})
+}
 
 const isAdmin = computed(() => {
 	return (
@@ -80,6 +71,15 @@ const isAdmin = computed(() => {
 	)
 })
 
+onMounted(() => {
+	if (isAdmin.value) {
+		currentTab.value = 'instructor'
+	} else {
+		currentTab.value = 'student'
+		fetchEvalCount()
+	}
+})
+
 const myLiveClasses = createResource({
 	url: 'lms.lms.api.get_my_live_classes',
 	auto: !isAdmin.value ? true : false,
@@ -151,11 +151,6 @@ const subtitle = computed(() => {
 	}
 })
 
-const tabs = [
-	{ label: __('Student'), value: 'student' },
-	{ label: __('Instructor'), value: 'instructor' },
-]
-
 usePageMeta(() => {
 	return {
 		title: __('Home'),
diff --git a/lms/lms/doctype/course_evaluator/course_evaluator.json b/lms/lms/doctype/course_evaluator/course_evaluator.json
index 5ffae917..41bd9f8c 100644
--- a/lms/lms/doctype/course_evaluator/course_evaluator.json
+++ b/lms/lms/doctype/course_evaluator/course_evaluator.json
@@ -83,7 +83,7 @@
  ],
  "index_web_pages_for_search": 1,
  "links": [],
- "modified": "2025-11-10 11:41:51.802016",
+ "modified": "2026-02-23 14:50:11.733278",
  "modified_by": "sayali@frappe.io",
  "module": "LMS",
  "name": "Course Evaluator",
@@ -125,6 +125,18 @@
    "role": "Batch Evaluator",
    "share": 1,
    "write": 1
+  },
+  {
+   "create": 1,
+   "delete": 1,
+   "email": 1,
+   "export": 1,
+   "print": 1,
+   "read": 1,
+   "report": 1,
+   "role": "Course Creator",
+   "share": 1,
+   "write": 1
   }
  ],
  "row_format": "Dynamic",
diff --git a/lms/lms/doctype/lms_certificate_request/lms_certificate_request.json b/lms/lms/doctype/lms_certificate_request/lms_certificate_request.json
index bf81aa55..a95fdff0 100644
--- a/lms/lms/doctype/lms_certificate_request/lms_certificate_request.json
+++ b/lms/lms/doctype/lms_certificate_request/lms_certificate_request.json
@@ -157,7 +157,7 @@
  "grid_page_length": 50,
  "index_web_pages_for_search": 1,
  "links": [],
- "modified": "2026-02-19 16:01:43.810407",
+ "modified": "2026-02-23 14:45:44.994705",
  "modified_by": "sayali@frappe.io",
  "module": "LMS",
  "name": "LMS Certificate Request",
@@ -210,6 +210,18 @@
    "role": "Batch Evaluator",
    "share": 1,
    "write": 1
+  },
+  {
+   "create": 1,
+   "delete": 1,
+   "email": 1,
+   "export": 1,
+   "print": 1,
+   "read": 1,
+   "report": 1,
+   "role": "Course Creator",
+   "share": 1,
+   "write": 1
   }
  ],
  "row_format": "Dynamic",