From c7915e2c3d0d10bf4f5b6bbd5f1ede836b17a48b Mon Sep 17 00:00:00 2001
From: Jannat Patel <pateljannat2308@gmail.com>
Date: Mon, 17 Nov 2025 10:10:26 +0530
Subject: [PATCH 01/14] feat: launch command palette

---
 frontend/components.d.ts                      |  1 +
 frontend/src/components/AppSidebar.vue        | 49 +++++++++++++------
 .../CommandPalette/CommandPalette.vue         | 10 ++++
 frontend/src/stores/settings.js               |  2 +
 frontend/src/utils/index.js                   |  5 ++
 5 files changed, 52 insertions(+), 15 deletions(-)
 create mode 100644 frontend/src/components/CommandPalette/CommandPalette.vue

diff --git a/frontend/components.d.ts b/frontend/components.d.ts
index 374bf990..fd58e52a 100644
--- a/frontend/components.d.ts
+++ b/frontend/components.d.ts
@@ -42,6 +42,7 @@ declare module 'vue' {
     CodeEditor: typeof import('./src/components/Controls/CodeEditor.vue')['default']
     CollapseSidebar: typeof import('./src/components/Icons/CollapseSidebar.vue')['default']
     ColorSwatches: typeof import('./src/components/Controls/ColorSwatches.vue')['default']
+    CommandPalette: typeof import('./src/components/CommandPalette/CommandPalette.vue')['default']
     ContactUsEmail: typeof import('./src/components/ContactUsEmail.vue')['default']
     CouponDetails: typeof import('./src/components/Settings/Coupons/CouponDetails.vue')['default']
     CouponItems: typeof import('./src/components/Settings/Coupons/CouponItems.vue')['default']
diff --git a/frontend/src/components/AppSidebar.vue b/frontend/src/components/AppSidebar.vue
index 9334c63e..31bd5e0b 100644
--- a/frontend/src/components/AppSidebar.vue
+++ b/frontend/src/components/AppSidebar.vue
@@ -173,6 +173,7 @@
 			:currentStep="currentStep"
 		/>
 	</div>
+	<CommandPalette v-model="settingsStore.isCommandPaletteOpen" />
 	<PageModal
 		v-model="showPageModal"
 		v-model:reloadSidebar="sidebarSettings"
@@ -181,19 +182,6 @@
 </template>
 
 <script setup>
-import UserDropdown from '@/components/UserDropdown.vue'
-import CollapseSidebar from '@/components/Icons/CollapseSidebar.vue'
-import SidebarLink from '@/components/SidebarLink.vue'
-import {
-	ref,
-	onMounted,
-	inject,
-	watch,
-	reactive,
-	markRaw,
-	h,
-	onUnmounted,
-} from 'vue'
 import { getSidebarLinks } from '@/utils'
 import { usersStore } from '@/stores/user'
 import { sessionStore } from '@/stores/session'
@@ -204,7 +192,16 @@ import PageModal from '@/components/Modals/PageModal.vue'
 import { capture } from '@/telemetry'
 import LMSLogo from '@/components/Icons/LMSLogo.vue'
 import { useRouter } from 'vue-router'
-import InviteIcon from './Icons/InviteIcon.vue'
+import {
+	ref,
+	onMounted,
+	inject,
+	watch,
+	reactive,
+	markRaw,
+	h,
+	onUnmounted,
+} from 'vue'
 import {
 	BookOpen,
 	CircleAlert,
@@ -217,7 +214,6 @@ import {
 	Users,
 	BookText,
 	Zap,
-	Check,
 } from 'lucide-vue-next'
 import {
 	TrialBanner,
@@ -228,6 +224,11 @@ import {
 	minimize,
 	IntermediateStepModal,
 } from 'frappe-ui/frappe'
+import InviteIcon from './Icons/InviteIcon.vue'
+import UserDropdown from '@/components/UserDropdown.vue'
+import CollapseSidebar from '@/components/Icons/CollapseSidebar.vue'
+import SidebarLink from '@/components/SidebarLink.vue'
+import CommandPalette from '@/components/CommandPalette/CommandPalette.vue'
 
 const { user } = sessionStore()
 const { userResource } = usersStore()
@@ -258,6 +259,7 @@ onMounted(() => {
 	addNotifications()
 	setSidebarLinks()
 	setUpOnboarding()
+	addKeyboardShortcut()
 	socket.on('publish_lms_notifications', (data) => {
 		unreadNotifications.reload()
 	})
@@ -280,6 +282,23 @@ const setSidebarLinks = () => {
 	)
 }
 
+const addKeyboardShortcut = () => {
+	window.addEventListener('keydown', (e) => {
+		if (
+			e.key === 'k' &&
+			(e.ctrlKey || e.metaKey) &&
+			!e.target.classList.contains('ProseMirror')
+		) {
+			toggleCommandPalette()
+			e.preventDefault()
+		}
+	})
+}
+
+const toggleCommandPalette = () => {
+	settingsStore.isCommandPaletteOpen = !settingsStore.isCommandPaletteOpen
+}
+
 const unreadNotifications = createResource({
 	cache: 'Unread Notifications Count',
 	url: 'frappe.client.get_count',
diff --git a/frontend/src/components/CommandPalette/CommandPalette.vue b/frontend/src/components/CommandPalette/CommandPalette.vue
new file mode 100644
index 00000000..25bc3d3e
--- /dev/null
+++ b/frontend/src/components/CommandPalette/CommandPalette.vue
@@ -0,0 +1,10 @@
+<template>
+	<Dialog v-model="show">
+		<template #body> Command Palette </template>
+	</Dialog>
+</template>
+<script setup lang="ts">
+import { Dialog } from 'frappe-ui'
+
+const show = defineModel<boolean>({ required: true, default: false })
+</script>
diff --git a/frontend/src/stores/settings.js b/frontend/src/stores/settings.js
index 539a292f..e8b4ba70 100644
--- a/frontend/src/stores/settings.js
+++ b/frontend/src/stores/settings.js
@@ -5,6 +5,7 @@ import { sessionStore } from './session'
 
 export const useSettings = defineStore('settings', () => {
 	const isSettingsOpen = ref(false)
+	const isCommandPaletteOpen = ref(false)
 	const activeTab = ref(null)
 
 	const allowGuestAccess = createResource({
@@ -50,6 +51,7 @@ export const useSettings = defineStore('settings', () => {
 
 	return {
 		isSettingsOpen,
+		isCommandPaletteOpen,
 		activeTab,
 		allowGuestAccess,
 		preventSkippingVideos,
diff --git a/frontend/src/utils/index.js b/frontend/src/utils/index.js
index fb24c9f7..13cb4e7e 100644
--- a/frontend/src/utils/index.js
+++ b/frontend/src/utils/index.js
@@ -403,6 +403,11 @@ export function getUserTimezone() {
 
 export function getSidebarLinks() {
 	return [
+		{
+			label: 'Search',
+			icon: 'Search',
+			to: 'Search',
+		},
 		{
 			label: 'Courses',
 			icon: 'BookOpen',

From eab43a66cfdef3b0e8a631ac3c52931f585efef8 Mon Sep 17 00:00:00 2001
From: Jannat Patel <pateljannat2308@gmail.com>
Date: Tue, 25 Nov 2025 19:44:31 +0530
Subject: [PATCH 02/14] feat: improved search results in command palette

---
 frappe-ui                                     |   2 +-
 frontend/components.d.ts                      |   4 +
 .../CommandPalette/CommandPalette.vue         | 249 +++++++++++++++++-
 .../CommandPalette/CommandPaletteGroup.vue    |  44 ++++
 frontend/src/pages/Search/Search.vue          |  11 +
 frontend/src/router.js                        |   5 +
 lms/command_palette.py                        |  29 ++
 lms/hooks.py                                  |   8 +
 lms/lms/doctype/lms_course/lms_course.json    |   6 +-
 lms/sqlite.py                                 | 122 +++++++++
 pyproject.toml                                |   2 +-
 11 files changed, 474 insertions(+), 8 deletions(-)
 create mode 100644 frontend/src/components/CommandPalette/CommandPaletteGroup.vue
 create mode 100644 frontend/src/pages/Search/Search.vue
 create mode 100644 lms/command_palette.py
 create mode 100644 lms/sqlite.py

diff --git a/frappe-ui b/frappe-ui
index 8d5956c0..204333c9 160000
--- a/frappe-ui
+++ b/frappe-ui
@@ -1 +1 @@
-Subproject commit 8d5956c0c675a13d32d3ef8035389128fa4735d5
+Subproject commit 204333c9256f21fca5f5c50acd66cee11aeca4f3
diff --git a/frontend/components.d.ts b/frontend/components.d.ts
index fd58e52a..b52f8116 100644
--- a/frontend/components.d.ts
+++ b/frontend/components.d.ts
@@ -43,6 +43,7 @@ declare module 'vue' {
     CollapseSidebar: typeof import('./src/components/Icons/CollapseSidebar.vue')['default']
     ColorSwatches: typeof import('./src/components/Controls/ColorSwatches.vue')['default']
     CommandPalette: typeof import('./src/components/CommandPalette/CommandPalette.vue')['default']
+    CommandPaletteGroup: typeof import('./src/components/CommandPalette/CommandPaletteGroup.vue')['default']
     ContactUsEmail: typeof import('./src/components/ContactUsEmail.vue')['default']
     CouponDetails: typeof import('./src/components/Settings/Coupons/CouponDetails.vue')['default']
     CouponItems: typeof import('./src/components/Settings/Coupons/CouponItems.vue')['default']
@@ -85,6 +86,9 @@ declare module 'vue' {
     LiveClassAttendance: typeof import('./src/components/Modals/LiveClassAttendance.vue')['default']
     LiveClassModal: typeof import('./src/components/Modals/LiveClassModal.vue')['default']
     LMSLogo: typeof import('./src/components/Icons/LMSLogo.vue')['default']
+    LucideArrowDown: typeof import('~icons/lucide/arrow-down')['default']
+    LucideArrowUp: typeof import('~icons/lucide/arrow-up')['default']
+    LucideCornerDownLeft: typeof import('~icons/lucide/corner-down-left')['default']
     Members: typeof import('./src/components/Settings/Members.vue')['default']
     MobileLayout: typeof import('./src/components/MobileLayout.vue')['default']
     MultiSelect: typeof import('./src/components/Controls/MultiSelect.vue')['default']
diff --git a/frontend/src/components/CommandPalette/CommandPalette.vue b/frontend/src/components/CommandPalette/CommandPalette.vue
index 25bc3d3e..c4fdf036 100644
--- a/frontend/src/components/CommandPalette/CommandPalette.vue
+++ b/frontend/src/components/CommandPalette/CommandPalette.vue
@@ -1,10 +1,253 @@
 <template>
-	<Dialog v-model="show">
-		<template #body> Command Palette </template>
+	<Dialog v-model="show" :options="{ size: '2xl' }">
+		<template #body>
+			<div class="text-base">
+				<div class="flex items-center space-x-2 pl-4.5 border-b">
+					<Search class="size-4 text-ink-gray-4" />
+					<input
+						ref="inputRef"
+						type="text"
+						placeholder="Search"
+						class="w-full border-none bg-transparent py-3 !pl-2 pr-4.5 text-base text-ink-gray-7 placeholder-ink-gray-4 focus:ring-0"
+						@input="onInput"
+						@keydown="onKeyDown"
+						v-model="query"
+						autocomplete="off"
+					/>
+				</div>
+
+				<div class="max-h-96 overflow-auto mb-2">
+					<div v-if="query.length" class="mt-5 space-y-5">
+						<CommandPaletteGroup :list="searchResults" />
+					</div>
+
+					<div v-else class="mt-5 space-y-5">
+						<CommandPaletteGroup :list="jumpToOptions" />
+					</div>
+				</div>
+
+				<div
+					class="flex items-center space-x-5 w-full border-t py-2 text-sm text-ink-gray-7 px-4.5"
+				>
+					<div class="flex items-center space-x-2">
+						<MoveUp
+							class="size-5 stroke-1.5 bg-surface-gray-2 p-1 rounded-sm"
+						/>
+						<MoveDown
+							class="size-5 stroke-1.5 bg-surface-gray-2 p-1 rounded-sm"
+						/>
+						<span>
+							{{ __('to navigate') }}
+						</span>
+					</div>
+					<div class="flex items-center space-x-2">
+						<CornerDownLeft
+							class="size-5 stroke-1.5 bg-surface-gray-2 p-1 rounded-sm"
+						/>
+						<span>
+							{{ __('to select') }}
+						</span>
+					</div>
+					<div class="flex items-center space-x-2">
+						<span class="bg-surface-gray-2 p-1 rounded-sm"> esc </span>
+						<span>
+							{{ __('to close') }}
+						</span>
+					</div>
+				</div>
+			</div>
+		</template>
 	</Dialog>
 </template>
 <script setup lang="ts">
-import { Dialog } from 'frappe-ui'
+import { createResource, debounce, Dialog } from 'frappe-ui'
+import { computed, inject, nextTick, onMounted, ref, watch } from 'vue'
+import {
+	BookOpen,
+	Briefcase,
+	CornerDownLeft,
+	FileSearch,
+	MoveUp,
+	MoveDown,
+	Search,
+	Users,
+} from 'lucide-vue-next'
+import { useRouter } from 'vue-router'
+import CommandPaletteGroup from './CommandPaletteGroup.vue'
 
 const show = defineModel<boolean>({ required: true, default: false })
+const router = useRouter()
+const query = ref<string>('')
+const searchResults = ref<Array<any>>([])
+
+const search = createResource({
+	url: 'lms.command_palette.search_sqlite',
+	makeParams: () => ({
+		query: query.value,
+	}),
+	onSuccess() {
+		generateSearchResults()
+	},
+})
+
+const debouncedSearch = debounce(() => {
+	if (query.value.length > 2) {
+		search.reload()
+	}
+}, 500)
+
+const onInput = () => {
+	debouncedSearch()
+}
+
+const generateSearchResults = () => {
+	search.data?.forEach((type: any) => {
+		let result = {}
+		result.title = type.title
+		type.items.forEach((item: any) => {
+			let paramName = item.doctype === 'LMS Course' ? 'courseName' : 'batchName'
+			item.route = {
+				name: item.doctype === 'LMS Course' ? 'CourseDetail' : 'BatchDetail',
+				params: {
+					[paramName]: item.name,
+				},
+			}
+			item.isActive = false
+		})
+		result.items = type.items
+		searchResults.value.push(result)
+	})
+}
+
+const appendSearchPage = () => {
+	let searchPage = {}
+	searchPage.title = __('Jump to')
+	searchPage.items = [
+		{
+			title: __('Search for ') + `"${query.value}"`,
+			route: {
+				name: 'Search',
+				query: {
+					q: query.value,
+				},
+			},
+			icon: FileSearch,
+			isActive: true,
+		},
+	]
+	searchResults.value = [searchPage]
+}
+
+watch(
+	query,
+	() => {
+		appendSearchPage()
+	},
+	{ immediate: true }
+)
+
+onMounted(() => {
+	addKeyboardShortcuts()
+})
+
+const addKeyboardShortcuts = () => {
+	window.addEventListener('keydown', (e: KeyboardEvent) => {
+		if (e.key === 'ArrowUp' && show.value) {
+			e.preventDefault()
+			shortcutForArrowKey(-1)
+		} else if (e.key === 'ArrowDown' && show.value) {
+			shortcutForArrowKey(1)
+		} else if (e.key === 'Enter' && show.value) {
+			shortcutForEnter()
+		} else if (e.key === 'Escape' && show.value) {
+			show.value = false
+		}
+	})
+}
+
+const shortcutForArrowKey = (direction: number) => {
+	let currentList = query.value.length
+		? searchResults.value
+		: jumpToOptions.value
+	let allItems = currentList.flatMap((result: any) => result.items)
+	let indexOfActive = allItems.findIndex((option: any) => option.isActive)
+	let newIndex = indexOfActive + direction
+	if (newIndex < 0) newIndex = allItems.length - 1
+	if (newIndex >= allItems.length) newIndex = 0
+	allItems[indexOfActive].isActive = false
+	allItems[newIndex].isActive = true
+	nextTick(scrollActiveItemIntoView)
+}
+
+const scrollActiveItemIntoView = () => {
+	const activeItem = document.querySelector(
+		'.hover\\:bg-surface-gray-2.bg-surface-gray-2'
+	) as HTMLElement
+	if (activeItem) {
+		activeItem.scrollIntoView({ block: 'nearest' })
+	}
+}
+
+const shortcutForEnter = () => {
+	let currentList = query.value.length
+		? searchResults.value
+		: jumpToOptions.value
+	let allItems = currentList.flatMap((result: any) => result.items)
+	let activeOption = allItems.find((option) => option.isActive)
+	if (activeOption) {
+		navigateTo(activeOption.route)
+	}
+}
+
+const navigateTo = (route: string) => {
+	show.value = false
+	query.value = ''
+	router.push({ name: route.name, params: route.params, query: route.query })
+}
+
+const jumpToOptions = ref([
+	{
+		title: __('Jump to'),
+		items: [
+			{
+				title: 'Advanced Search',
+				icon: Search,
+				route: {
+					name: 'Search',
+				},
+				isActive: true,
+			},
+			{
+				title: 'Courses',
+				icon: BookOpen,
+				route: {
+					name: 'Courses',
+				},
+				isActive: false,
+			},
+			{
+				title: 'Batches',
+				icon: Users,
+				route: {
+					name: 'Batches',
+				},
+				isActive: false,
+			},
+			{
+				title: 'Jobs',
+				icon: Briefcase,
+				route: {
+					name: 'Jobs',
+				},
+				isActive: false,
+			},
+		],
+	},
+])
 </script>
+<style>
+mark {
+	background-color: theme('colors.amber.100');
+	font-weight: 500;
+}
+</style>
diff --git a/frontend/src/components/CommandPalette/CommandPaletteGroup.vue b/frontend/src/components/CommandPalette/CommandPaletteGroup.vue
new file mode 100644
index 00000000..cf34b593
--- /dev/null
+++ b/frontend/src/components/CommandPalette/CommandPaletteGroup.vue
@@ -0,0 +1,44 @@
+<template>
+	<div v-for="result in list" class="px-2.5 space-y-2">
+		<div class="text-ink-gray-5 px-2">
+			{{ result.title }}
+		</div>
+		<div class="">
+			<div
+				v-for="item in result.items"
+				class="flex items-center justify-between p-2 rounded hover:bg-surface-gray-2 cursor-pointer"
+				:class="{ 'bg-surface-gray-2': item.isActive }"
+			>
+				<div class="flex items-center space-x-3">
+					<component
+						v-if="item.icon"
+						:is="item.icon"
+						class="size-4 stroke-1.5 text-ink-gray-6"
+					/>
+					<div v-html="item.title"></div>
+				</div>
+				<div v-if="item.modified" class="text-ink-gray-5">
+					{{ item.modified }} {{ dayjs.unix(item.modified).fromNow(true) }}
+				</div>
+			</div>
+		</div>
+	</div>
+</template>
+<script lang="ts" setup>
+import { inject } from 'vue'
+import { dayjsLocal } from 'frappe-ui'
+
+const dayjs = inject<any>('$dayjs')
+
+const props = defineProps<{
+	list: Array<{
+		title: string
+		items: Array<{
+			title: string
+			icon?: any
+			isActive?: boolean
+			modified?: string
+		}>
+	}>
+}>()
+</script>
diff --git a/frontend/src/pages/Search/Search.vue b/frontend/src/pages/Search/Search.vue
new file mode 100644
index 00000000..bc577e7b
--- /dev/null
+++ b/frontend/src/pages/Search/Search.vue
@@ -0,0 +1,11 @@
+<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: __('Search') }]" />
+	</header>
+	<div></div>
+</template>
+<script setup lang="ts">
+import { Breadcrumbs, TextInput } from 'frappe-ui'
+</script>
diff --git a/frontend/src/router.js b/frontend/src/router.js
index 9b0234f8..c9fba46e 100644
--- a/frontend/src/router.js
+++ b/frontend/src/router.js
@@ -243,6 +243,11 @@ const routes = [
 			),
 		props: true,
 	},
+	{
+		path: '/search',
+		name: 'Search',
+		component: () => import('@/pages/Search/Search.vue'),
+	},
 ]
 
 let router = createRouter({
diff --git a/lms/command_palette.py b/lms/command_palette.py
new file mode 100644
index 00000000..077c9697
--- /dev/null
+++ b/lms/command_palette.py
@@ -0,0 +1,29 @@
+import frappe
+
+
+@frappe.whitelist()
+def search_sqlite(query: str):
+	from lms.sqlite import LearningSearch, LearningSearchIndexMissingError
+
+	search = LearningSearch()
+
+	try:
+		result = search.search(query)
+	except LearningSearchIndexMissingError:
+		return []
+
+	groups = {}
+	print(result)
+	for r in result["results"]:
+		doctype = r["doctype"]
+
+		if doctype == "LMS Course":
+			groups.setdefault("Courses", []).append(r)
+		elif doctype == "LMS Batch":
+			groups.setdefault("Batches", []).append(r)
+
+	out = []
+	for key in groups:
+		out.append({"title": key, "items": groups[key]})
+
+	return out
diff --git a/lms/hooks.py b/lms/hooks.py
index 038eee9d..361aa3b2 100644
--- a/lms/hooks.py
+++ b/lms/hooks.py
@@ -64,6 +64,9 @@ after_install = "lms.install.after_install"
 after_sync = "lms.install.after_sync"
 before_uninstall = "lms.install.before_uninstall"
 setup_wizard_requires = "assets/lms/js/setup_wizard.js"
+after_migrate = [
+	"lms.sqlite.build_index_in_background",
+]
 
 # Desk Notifications
 # ------------------
@@ -112,6 +115,9 @@ doc_events = {
 # Scheduled Tasks
 # ---------------
 scheduler_events = {
+	"all": [
+		"lms.sqlite.build_index_in_background",
+	],
 	"hourly": [
 		"lms.lms.doctype.lms_certificate_request.lms_certificate_request.schedule_evals",
 		"lms.lms.api.update_course_statistics",
@@ -251,3 +257,5 @@ add_to_apps_screen = [
 		"has_permission": "lms.lms.api.check_app_permission",
 	}
 ]
+
+sqlite_search = ["lms.sqlite.LearningSearch"]
diff --git a/lms/lms/doctype/lms_course/lms_course.json b/lms/lms/doctype/lms_course/lms_course.json
index 20fc0482..b30ed240 100644
--- a/lms/lms/doctype/lms_course/lms_course.json
+++ b/lms/lms/doctype/lms_course/lms_course.json
@@ -76,6 +76,8 @@
    "default": "0",
    "fieldname": "published",
    "fieldtype": "Check",
+   "in_list_view": 1,
+   "in_standard_filter": 1,
    "label": "Published"
   },
   {
@@ -152,8 +154,6 @@
    "fieldname": "status",
    "fieldtype": "Select",
    "hidden": 1,
-   "in_list_view": 1,
-   "in_standard_filter": 1,
    "label": "Status",
    "options": "In Progress\nUnder Review\nApproved",
    "read_only": 1
@@ -313,7 +313,7 @@
   }
  ],
  "make_attachments_public": 1,
- "modified": "2025-10-13 15:08:11.734204",
+ "modified": "2025-11-25 11:35:17.924569",
  "modified_by": "sayali@frappe.io",
  "module": "LMS",
  "name": "LMS Course",
diff --git a/lms/sqlite.py b/lms/sqlite.py
new file mode 100644
index 00000000..f8f30e03
--- /dev/null
+++ b/lms/sqlite.py
@@ -0,0 +1,122 @@
+from contextlib import suppress
+
+import frappe
+from frappe.search.sqlite_search import SQLiteSearch, SQLiteSearchIndexMissingError
+from frappe.utils import update_progress_bar
+from redis.exceptions import ResponseError
+
+
+class LearningSearch(SQLiteSearch):
+	INDEX_NAME = "learning.db"
+
+	INDEX_SCHEMA = {
+		"metadata_fields": ["category", "owner", "published"],
+		"tokenizer": "unicode61 remove_diacritics 2 tokenchars '-_'",
+	}
+
+	INDEXABLE_DOCTYPES = {
+		"LMS Course": {
+			"fields": [
+				"name",
+				"title",
+				{"content": "description"},
+				"short_introduction",
+				"published",
+				"category",
+				"owner",
+				{"modified": "published_on"},
+			],
+		},
+		"LMS Batch": {
+			"fields": [
+				"name",
+				"title",
+				"description",
+				{"content": "batch_details"},
+				"published",
+				"category",
+				"owner",
+				{"modified": "start_date"},
+			],
+		},
+	}
+
+	DOCTYPE_FIELDS = {
+		"LMS Course": [
+			"name",
+			"title",
+			"description",
+			"short_introduction",
+			"category",
+			"creation",
+			"modified",
+			"owner",
+		],
+		"LMS Batch": [
+			"name",
+			"title",
+			"description",
+			"batch_details",
+			"category",
+			"creation",
+			"modified",
+			"owner",
+		],
+	}
+
+	def can_create_course(self, roles):
+		return "Course Creator" in roles or "Moderator" in roles
+
+	def can_create_batch(self, roles):
+		return "Batch Evaluator" in roles or "Moderator" in roles
+
+	def get_records(self, doctype):
+		records = []
+		roles = frappe.get_roles()
+		filters = {}
+
+		if doctype == "LMS Course":
+			if not self.can_create_course(roles):
+				filters = {"published": 1}
+
+		if doctype == "LMS Batch":
+			if not self.can_create_batch(roles):
+				filters = {"published": 1}
+
+		records = frappe.db.get_all(doctype, filters=filters, fields=self.DOCTYPE_FIELDS[doctype])
+		for record in records:
+			record["doctype"] = doctype
+
+		return records
+
+	def build_index(self):
+		try:
+			super().build_index()
+		except Exception as e:
+			frappe.throw(e)
+
+	def get_search_filters(self):
+		roles = frappe.get_roles()
+		if not (self.can_create_course(roles) and self.can_create_batch(roles)):
+			return {"published": 1}
+		return {}
+
+
+class LearningSearchIndexMissingError(SQLiteSearchIndexMissingError):
+	pass
+
+
+def build_index():
+	search = LearningSearch()
+	search.build_index()
+
+
+def build_index_in_background():
+	if not frappe.cache().get_value("learning_search_indexing_in_progress"):
+		frappe.enqueue(build_index, queue="long")
+
+
+def build_index_if_not_exists():
+	search = LearningSearch()
+	if not search.index_exists():
+		build_index()
diff --git a/pyproject.toml b/pyproject.toml
index fc16f51d..7d7f4193 100644
--- a/pyproject.toml
+++ b/pyproject.toml
@@ -11,7 +11,7 @@ dependencies = [
     # core dependencies
     "websocket_client~=1.6.4",
     "markdown~=3.5.1",
-    "beautifulsoup4~=4.12.2",
+    "beautifulsoup4~=4.13.4",
     "lxml~=4.9.3",
     "cairocffi==1.5.1",
     "razorpay~=1.4.1",

From d82517f402b6d9353eb1d3bfbb5d2aed2d452fdc Mon Sep 17 00:00:00 2001
From: Jannat Patel <pateljannat2308@gmail.com>
Date: Mon, 8 Dec 2025 20:22:55 +0530
Subject: [PATCH 03/14] test: utils

---
 .../components/Modals/JobApplicationModal.vue |   7 +-
 frontend/src/pages/Home/AdminHome.vue         |   4 +-
 frontend/src/pages/Home/Home.vue              |   6 +-
 frontend/src/pages/Home/StudentHome.vue       |   4 +-
 frontend/src/pages/JobDetail.vue              |  59 +-
 lms/hooks.py                                  |   1 -
 lms/lms/api.py                                | 512 ++++++++++-----
 .../exercise_latest_submission/__init__.py    |   0
 .../exercise_latest_submission.js             |   7 -
 .../exercise_latest_submission.json           | 166 -----
 .../exercise_latest_submission.py             |   9 -
 .../test_exercise_latest_submission.py        |   9 -
 .../doctype/exercise_submission/__init__.py   |   0
 .../exercise_submission.js                    |   7 -
 .../exercise_submission.json                  | 126 ----
 .../exercise_submission.py                    |  29 -
 .../test_exercise_submission.py               |   9 -
 lms/lms/doctype/lms_batch_old/__init__.py     |   0
 .../doctype/lms_batch_old/lms_batch_old.js    |   7 -
 .../doctype/lms_batch_old/lms_batch_old.json  | 150 -----
 .../doctype/lms_batch_old/lms_batch_old.py    |  90 ---
 .../lms_batch_old/test_lms_batch_old.py       |   9 -
 .../lms_certificate/lms_certificate.py        |  11 +-
 lms/lms/doctype/lms_course/lms_course.py      |  31 -
 lms/lms/doctype/lms_course/test_lms_course.py |  13 -
 .../lms_course_review/lms_course_review.py    |   9 +-
 .../lms_enrollment/lms_enrollment.json        |   8 +-
 .../doctype/lms_enrollment/lms_enrollment.py  |  28 -
 lms/lms/doctype/lms_exercise/__init__.py      |   0
 lms/lms/doctype/lms_exercise/lms_exercise.js  |   7 -
 .../doctype/lms_exercise/lms_exercise.json    | 123 ----
 lms/lms/doctype/lms_exercise/lms_exercise.py  |  52 --
 .../doctype/lms_exercise/test_lms_exercise.py |  10 -
 lms/lms/test_utils.py                         | 161 ++++-
 lms/lms/utils.py                              | 617 +-----------------
 lms/templates/livecode/extension_footer.html  | 168 -----
 lms/templates/livecode/extension_header.html  |   8 -
 37 files changed, 585 insertions(+), 1872 deletions(-)
 delete mode 100644 lms/lms/doctype/exercise_latest_submission/__init__.py
 delete mode 100644 lms/lms/doctype/exercise_latest_submission/exercise_latest_submission.js
 delete mode 100644 lms/lms/doctype/exercise_latest_submission/exercise_latest_submission.json
 delete mode 100644 lms/lms/doctype/exercise_latest_submission/exercise_latest_submission.py
 delete mode 100644 lms/lms/doctype/exercise_latest_submission/test_exercise_latest_submission.py
 delete mode 100644 lms/lms/doctype/exercise_submission/__init__.py
 delete mode 100644 lms/lms/doctype/exercise_submission/exercise_submission.js
 delete mode 100644 lms/lms/doctype/exercise_submission/exercise_submission.json
 delete mode 100644 lms/lms/doctype/exercise_submission/exercise_submission.py
 delete mode 100644 lms/lms/doctype/exercise_submission/test_exercise_submission.py
 delete mode 100644 lms/lms/doctype/lms_batch_old/__init__.py
 delete mode 100644 lms/lms/doctype/lms_batch_old/lms_batch_old.js
 delete mode 100644 lms/lms/doctype/lms_batch_old/lms_batch_old.json
 delete mode 100644 lms/lms/doctype/lms_batch_old/lms_batch_old.py
 delete mode 100644 lms/lms/doctype/lms_batch_old/test_lms_batch_old.py
 delete mode 100644 lms/lms/doctype/lms_exercise/__init__.py
 delete mode 100644 lms/lms/doctype/lms_exercise/lms_exercise.js
 delete mode 100644 lms/lms/doctype/lms_exercise/lms_exercise.json
 delete mode 100644 lms/lms/doctype/lms_exercise/lms_exercise.py
 delete mode 100644 lms/lms/doctype/lms_exercise/test_lms_exercise.py
 delete mode 100644 lms/templates/livecode/extension_footer.html
 delete mode 100644 lms/templates/livecode/extension_header.html

diff --git a/frontend/src/components/Modals/JobApplicationModal.vue b/frontend/src/components/Modals/JobApplicationModal.vue
index 06072243..b5124596 100644
--- a/frontend/src/components/Modals/JobApplicationModal.vue
+++ b/frontend/src/components/Modals/JobApplicationModal.vue
@@ -17,7 +17,7 @@
 		}"
 	>
 		<template #body-content>
-			<div class="flex flex-col gap-4">
+			<div class="flex flex-col gap-4 text-base">
 				<p class="text-ink-gray-9">
 					{{
 						__(
@@ -39,6 +39,9 @@
 						<template v-slot="{ file, progress, uploading, openFileSelector }">
 							<div class="">
 								<Button @click="openFileSelector" :loading="uploading">
+									<template #prefix>
+										<Upload class="size-4 stroke-1.5" />
+									</template>
 									{{
 										uploading ? `Uploading ${progress}%` : 'Upload your resume'
 									}}
@@ -66,7 +69,7 @@
 </template>
 <script setup>
 import { Dialog, FileUploader, Button, createResource, toast } from 'frappe-ui'
-import { FileText } from 'lucide-vue-next'
+import { FileText, Upload } from 'lucide-vue-next'
 import { ref, inject } from 'vue'
 import { getFileSize } from '@/utils/'
 
diff --git a/frontend/src/pages/Home/AdminHome.vue b/frontend/src/pages/Home/AdminHome.vue
index 04cfef82..e4c6dd22 100644
--- a/frontend/src/pages/Home/AdminHome.vue
+++ b/frontend/src/pages/Home/AdminHome.vue
@@ -222,12 +222,12 @@ const props = defineProps<{
 }>()
 
 const createdCourses = createResource({
-	url: 'lms.lms.utils.get_created_courses',
+	url: 'lms.lms.api.get_created_courses',
 	auto: true,
 })
 
 const createdBatches = createResource({
-	url: 'lms.lms.utils.get_created_batches',
+	url: 'lms.lms.api.get_created_batches',
 	auto: true,
 })
 
diff --git a/frontend/src/pages/Home/Home.vue b/frontend/src/pages/Home/Home.vue
index 54543bb2..dce42bb9 100644
--- a/frontend/src/pages/Home/Home.vue
+++ b/frontend/src/pages/Home/Home.vue
@@ -79,17 +79,17 @@ const myLiveClasses = createResource({
 })
 
 const adminLiveClasses = createResource({
-	url: 'lms.lms.utils.get_admin_live_classes',
+	url: 'lms.lms.api.get_admin_live_classes',
 	auto: isAdmin.value ? true : false,
 })
 
 const adminEvals = createResource({
-	url: 'lms.lms.utils.get_admin_evals',
+	url: 'lms.lms.api.get_admin_evals',
 	auto: isAdmin.value ? true : false,
 })
 
 const streakInfo = createResource({
-	url: 'lms.lms.utils.get_streak_info',
+	url: 'lms.lms.api.get_streak_info',
 	auto: true,
 })
 
diff --git a/frontend/src/pages/Home/StudentHome.vue b/frontend/src/pages/Home/StudentHome.vue
index 951917a8..baab0447 100644
--- a/frontend/src/pages/Home/StudentHome.vue
+++ b/frontend/src/pages/Home/StudentHome.vue
@@ -161,12 +161,12 @@ const props = defineProps<{
 }>()
 
 const myCourses = createResource({
-	url: 'lms.lms.utils.get_my_courses',
+	url: 'lms.lms.api.get_my_courses',
 	auto: true,
 })
 
 const myBatches = createResource({
-	url: 'lms.lms.utils.get_my_batches',
+	url: 'lms.lms.api.get_my_batches',
 	auto: true,
 })
 
diff --git a/frontend/src/pages/JobDetail.vue b/frontend/src/pages/JobDetail.vue
index 9987bb42..b740ded8 100644
--- a/frontend/src/pages/JobDetail.vue
+++ b/frontend/src/pages/JobDetail.vue
@@ -11,8 +11,8 @@
 						route: { name: 'Jobs' },
 					},
 					{
-						label: job.data?.job_title,
-						route: { name: 'JobDetail', params: { job: job.data?.name } },
+						label: job.doc?.job_title,
+						route: { name: 'JobDetail', params: { job: job.doc?.name } },
 					},
 				]"
 			/>
@@ -24,7 +24,7 @@
 					v-if="canManageJob && applicationCount.data > 0"
 					:to="{
 						name: 'JobApplications',
-						params: { job: job.data?.name },
+						params: { job: job.doc?.name },
 					}"
 				>
 					<Button variant="subtle">
@@ -35,7 +35,7 @@
 					v-if="canManageJob"
 					:to="{
 						name: 'JobForm',
-						params: { jobName: job.data?.name },
+						params: { jobName: job.doc?.name },
 					}"
 				>
 					<Button>
@@ -45,7 +45,7 @@
 						{{ __('Edit') }}
 					</Button>
 				</router-link>
-				<Button @click="redirectToWebsite(job.data?.company_website)">
+				<Button @click="redirectToWebsite(job.doc?.company_website)">
 					<template #prefix>
 						<SquareArrowOutUpRight class="h-4 w-4 stroke-1.5" />
 					</template>
@@ -69,30 +69,30 @@
 				</Badge>
 			</div>
 			<div v-else-if="!readOnlyMode">
-				<Button @click="redirectToLogin(job.data?.name)">
+				<Button @click="redirectToLogin(job.doc?.name)">
 					<span>
 						{{ __('Login to apply') }}
 					</span>
 				</Button>
 			</div>
 		</header>
-		<div v-if="job.data" class="max-w-3xl mx-auto pt-5">
+		<div v-if="job.doc" class="max-w-3xl mx-auto pt-5">
 			<div class="p-4">
 				<div class="space-y-5 mb-12">
 					<div class="flex">
 						<img
-							:src="job.data.company_logo"
+							:src="job.doc.company_logo"
 							class="size-10 rounded-lg object-contain cursor-pointer mr-4"
-							:alt="job.data.company_name"
-							@click="redirectToWebsite(job.data.company_website)"
+							:alt="job.doc.company_name"
+							@click="redirectToWebsite(job.doc.company_website)"
 						/>
 						<div class="">
 							<div class="text-2xl text-ink-gray-9 font-semibold mb-1">
-								{{ job.data.job_title }}
+								{{ job.doc.job_title }}
 							</div>
 							<div class="text-sm text-ink-gray-5 font-semibold">
-								{{ job.data.company_name }} - {{ job.data.location }},
-								{{ job.data.country }}
+								{{ job.doc.company_name }} - {{ job.doc.location }},
+								{{ job.doc.country }}
 							</div>
 						</div>
 					</div>
@@ -102,19 +102,19 @@
 							<template #prefix>
 								<CalendarDays class="size-3 stroke-2 text-ink-gray-7" />
 							</template>
-							{{ dayjs(job.data.creation).fromNow() }}
+							{{ dayjs(job.doc.creation).fromNow() }}
 						</Badge>
 						<Badge size="lg">
 							<template #prefix>
 								<ClipboardType class="size-3 stroke-2 text-ink-gray-7" />
 							</template>
-							{{ job.data.type }}
+							{{ job.doc.type }}
 						</Badge>
-						<Badge v-if="job.data?.work_mode" size="lg">
+						<Badge v-if="job.doc?.work_mode" size="lg">
 							<template #prefix>
 								<BriefcaseBusiness class="size-3 stroke-2 text-ink-gray-7" />
 							</template>
-							{{ job.data.work_mode }}
+							{{ job.doc.work_mode }}
 						</Badge>
 						<Badge v-if="applicationCount.data" size="lg">
 							<template #prefix>
@@ -137,14 +137,14 @@
 				</div>
 
 				<p
-					v-html="job.data.description"
+					v-html="job.doc.description"
 					class="ProseMirror prose prose-table:table-fixed prose-td:p-2 prose-th:p-2 prose-td:border prose-th:border prose-td:border-outline-gray-2 prose-th:border-outline-gray-2 prose-td:relative prose-th:relative prose-th:bg-surface-gray-2 prose-sm max-w-none !whitespace-normal mt-12"
 				></p>
 			</div>
 			<JobApplicationModal
 				v-model="showApplicationModal"
 				v-model:application="jobApplication"
-				:job="job.data.name"
+				:job="job.doc.name"
 			/>
 		</div>
 	</div>
@@ -155,6 +155,7 @@ import {
 	Button,
 	Breadcrumbs,
 	createResource,
+	createDocumentResource,
 	usePageMeta,
 } from 'frappe-ui'
 import { inject, ref, computed } from 'vue'
@@ -186,13 +187,11 @@ const props = defineProps({
 	},
 })
 
-const job = createResource({
-	url: 'lms.lms.api.get_job_details',
-	params: {
-		job: props.job,
-	},
-	cache: ['job', props.job],
+const job = createDocumentResource({
+	doctype: 'Job Opportunity',
+	name: props.job,
 	auto: true,
+	cache: ['job', props.job],
 	onSuccess: (data) => {
 		if (user.data?.name) {
 			jobApplication.submit()
@@ -207,7 +206,7 @@ const jobApplication = createResource({
 		return {
 			doctype: 'LMS Job Application',
 			filters: {
-				job: job.data?.name,
+				job: job.doc?.name,
 				user: user.data?.name,
 			},
 		}
@@ -220,7 +219,7 @@ const applicationCount = createResource({
 		return {
 			doctype: 'LMS Job Application',
 			filters: {
-				job: job.data?.name,
+				job: job.doc?.name,
 			},
 		}
 	},
@@ -239,13 +238,13 @@ const redirectToWebsite = (url) => {
 }
 
 const canManageJob = computed(() => {
-	if (!user.data?.name || !job.data) return false
-	return user.data.name === job.data.owner || user.data?.is_moderator
+	if (!user.data?.name || !job.doc) return false
+	return user.data.name === job.doc.owner || user.data?.is_moderator
 })
 
 usePageMeta(() => {
 	return {
-		title: job.data?.job_title,
+		title: job.doc?.job_title,
 		icon: brand.favicon,
 	}
 })
diff --git a/lms/hooks.py b/lms/hooks.py
index 4e61b7fe..7ac0e030 100644
--- a/lms/hooks.py
+++ b/lms/hooks.py
@@ -191,7 +191,6 @@ update_website_context = [
 
 jinja = {
 	"methods": [
-		"lms.lms.utils.get_signup_optin_checks",
 		"lms.lms.utils.get_tags",
 		"lms.lms.utils.get_lesson_count",
 		"lms.lms.utils.get_instructors",
diff --git a/lms/lms/api.py b/lms/lms/api.py
index 66fb97c8..e8ab5721 100644
--- a/lms/lms/api.py
+++ b/lms/lms/api.py
@@ -6,6 +6,7 @@ import re
 import shutil
 import xml.etree.ElementTree as ET
 import zipfile
+from datetime import timedelta
 from xml.dom.minidom import parseString
 
 import frappe
@@ -23,116 +24,13 @@ from frappe.utils import (
 	flt,
 	format_date,
 	get_datetime,
+	getdate,
 	now,
 )
 from frappe.utils.response import Response
 
 from lms.lms.doctype.course_lesson.course_lesson import save_progress
-from lms.lms.utils import get_average_rating, get_lesson_count
-
-
-@frappe.whitelist()
-def autosave_section(section, code):
-	"""Saves the code edited in one of the sections."""
-	doc = frappe.get_doc(doctype="Code Revision", section=section, code=code, author=frappe.session.user)
-	doc.insert()
-	return {"name": doc.name}
-
-
-@frappe.whitelist()
-def submit_solution(exercise, code):
-	"""Submits a solution.
-
-	@exerecise: name of the exercise to submit
-	@code: solution to the exercise
-	"""
-	ex = frappe.get_doc("LMS Exercise", exercise)
-	if not ex:
-		return
-	doc = ex.submit(code)
-	return {"name": doc.name, "creation": doc.creation}
-
-
-@frappe.whitelist()
-def save_current_lesson(course_name, lesson_name):
-	"""Saves the current lesson for a student/mentor."""
-	name = frappe.get_value(
-		doctype="LMS Enrollment",
-		filters={"course": course_name, "member": frappe.session.user},
-		fieldname="name",
-	)
-	if not name:
-		return
-	frappe.db.set_value("LMS Enrollment", name, "current_lesson", lesson_name)
-
-
-@frappe.whitelist()
-def join_cohort(course, cohort, subgroup, invite_code):
-	"""Creates a Cohort Join Request for given user."""
-	course_doc = frappe.get_doc("LMS Course", course)
-	cohort_doc = course_doc and course_doc.get_cohort(cohort)
-	subgroup_doc = cohort_doc and cohort_doc.get_subgroup(subgroup)
-
-	if not subgroup_doc or subgroup_doc.invite_code != invite_code:
-		return {"ok": False, "error": "Invalid join link"}
-
-	data = {
-		"doctype": "Cohort Join Request",
-		"cohort": cohort_doc.name,
-		"subgroup": subgroup_doc.name,
-		"email": frappe.session.user,
-		"status": "Pending",
-	}
-	# Don't insert duplicate records
-	if frappe.db.exists(data):
-		return {"ok": True, "status": "record found"}
-	else:
-		doc = frappe.get_doc(data)
-		doc.insert()
-		return {"ok": True, "status": "record created"}
-
-
-@frappe.whitelist()
-def approve_cohort_join_request(join_request):
-	r = frappe.get_doc("Cohort Join Request", join_request)
-	sg = r and frappe.get_doc("Cohort Subgroup", r.subgroup)
-	if not sg or r.status not in ["Pending", "Accepted"]:
-		return {"ok": False, "error": "Invalid Join Request"}
-	if not sg.is_manager(frappe.session.user) and "System Manager" not in frappe.get_roles():
-		return {"ok": False, "error": "Permission Deined"}
-
-	r.status = "Accepted"
-	r.save()
-	return {"ok": True}
-
-
-@frappe.whitelist()
-def reject_cohort_join_request(join_request):
-	r = frappe.get_doc("Cohort Join Request", join_request)
-	sg = r and frappe.get_doc("Cohort Subgroup", r.subgroup)
-	if not sg or r.status not in ["Pending", "Rejected"]:
-		return {"ok": False, "error": "Invalid Join Request"}
-	if not sg.is_manager(frappe.session.user) and "System Manager" not in frappe.get_roles():
-		return {"ok": False, "error": "Permission Deined"}
-
-	r.status = "Rejected"
-	r.save()
-	return {"ok": True}
-
-
-@frappe.whitelist()
-def undo_reject_cohort_join_request(join_request):
-	r = frappe.get_doc("Cohort Join Request", join_request)
-	sg = r and frappe.get_doc("Cohort Subgroup", r.subgroup)
-	# keeping Pending as well to consider the case of duplicate requests
-	if not sg or r.status not in ["Pending", "Rejected"]:
-		return {"ok": False, "error": "Invalid Join Request"}
-	if not sg.is_manager(frappe.session.user) and "System Manager" not in frappe.get_roles():
-		return {"ok": False, "error": "Permission Deined"}
-
-	r.status = "Pending"
-	r.save()
-	return {"ok": True}
+from lms.lms.utils import get_average_rating, get_batch_details, get_course_details, get_lesson_count
 
 
 @frappe.whitelist(allow_guest=True)
@@ -171,9 +69,32 @@ def get_translations():
 
 @frappe.whitelist()
 def validate_billing_access(billing_type, name):
+	doctype = "LMS Batch" if billing_type == "batch" else "LMS Course"
+	access, message = verify_billing_access(doctype, name, billing_type)
+
+	address = frappe.db.get_value(
+		"Address",
+		{"email_id": frappe.session.user},
+		[
+			"name",
+			"address_title as billing_name",
+			"address_line1",
+			"address_line2",
+			"city",
+			"state",
+			"country",
+			"pincode",
+			"phone",
+		],
+		as_dict=1,
+	)
+
+	return {"access": access, "message": message, "address": address}
+
+
+def verify_billing_access(doctype, name, billing_type):
 	access = True
 	message = ""
-	doctype = "LMS Batch" if billing_type == "batch" else "LMS Course"
 
 	if frappe.session.user == "Guest":
 		access = False
@@ -223,47 +144,7 @@ def validate_billing_access(billing_type, name):
 			access = False
 			message = _("You have already purchased the certificate for this course.")
 
-	address = frappe.db.get_value(
-		"Address",
-		{"email_id": frappe.session.user},
-		[
-			"name",
-			"address_title as billing_name",
-			"address_line1",
-			"address_line2",
-			"city",
-			"state",
-			"country",
-			"pincode",
-			"phone",
-		],
-		as_dict=1,
-	)
-
-	return {"access": access, "message": message, "address": address}
-
-
-@frappe.whitelist(allow_guest=True)
-def get_job_details(job):
-	return frappe.db.get_value(
-		"Job Opportunity",
-		job,
-		[
-			"job_title",
-			"location",
-			"country",
-			"type",
-			"work_mode",
-			"company_name",
-			"company_logo",
-			"company_website",
-			"name",
-			"creation",
-			"description",
-			"owner",
-		],
-		as_dict=1,
-	)
+	return access, message
 
 
 @frappe.whitelist(allow_guest=True)
@@ -1738,3 +1619,340 @@ def get_profile_details(username):
 
 	details.roles = frappe.get_roles(details.name)
 	return details
+
+
+@frappe.whitelist()
+def get_streak_info():
+	if frappe.session.user == "Guest":
+		return {}
+
+	all_dates = fetch_activity_dates(frappe.session.user)
+	streak, longest_streak = calculate_streaks(all_dates)
+	current_streak = calculate_current_streak(all_dates, streak)
+
+	return {
+		"current_streak": current_streak,
+		"longest_streak": longest_streak,
+	}
+
+
+def fetch_activity_dates(user):
+	doctypes = [
+		"LMS Course Progress",
+		"LMS Quiz Submission",
+		"LMS Assignment Submission",
+		"LMS Programming Exercise Submission",
+	]
+
+	all_dates = []
+	for dt in doctypes:
+		all_dates.extend(frappe.get_all(dt, {"member": user}, pluck="creation"))
+
+	return sorted({d.date() if hasattr(d, "date") else d for d in all_dates})
+
+
+def calculate_streaks(all_dates):
+	streak = 0
+	longest_streak = 0
+	prev_day = None
+
+	for d in all_dates:
+		if d.weekday() in (5, 6):
+			continue
+
+		if prev_day:
+			expected = prev_day + timedelta(days=1)
+			while expected.weekday() in (5, 6):
+				expected += timedelta(days=1)
+
+			streak = streak + 1 if d == expected else 1
+		else:
+			streak = 1
+
+		longest_streak = max(longest_streak, streak)
+		prev_day = d
+
+	return streak, longest_streak
+
+
+def calculate_current_streak(all_dates, streak):
+	if not all_dates:
+		return 0
+
+	last_date = all_dates[-1]
+	today = getdate()
+
+	ref_day = today
+	while ref_day.weekday() in (5, 6):
+		ref_day -= timedelta(days=1)
+
+	if last_date == ref_day or last_date == ref_day - timedelta(days=1):
+		return streak
+	return 0
+
+
+@frappe.whitelist()
+def get_my_live_classes():
+	my_live_classes = []
+	if frappe.session.user == "Guest":
+		return my_live_classes
+
+	batches = frappe.get_all(
+		"LMS Batch Enrollment",
+		{
+			"member": frappe.session.user,
+		},
+		order_by="creation desc",
+		pluck="batch",
+	)
+
+	live_class_details = frappe.get_all(
+		"LMS Live Class",
+		filters={
+			"date": [">=", getdate()],
+			"batch_name": ["in", batches],
+		},
+		fields=[
+			"name",
+			"title",
+			"description",
+			"time",
+			"date",
+			"duration",
+			"attendees",
+			"start_url",
+			"join_url",
+			"owner",
+		],
+		limit=2,
+		order_by="date",
+	)
+
+	if len(live_class_details):
+		for live_class in live_class_details:
+			live_class.course_title = frappe.db.get_value("LMS Course", live_class.course, "title")
+
+			my_live_classes.append(live_class)
+
+	return my_live_classes
+
+
+@frappe.whitelist()
+def get_created_courses():
+	created_courses = []
+	if frappe.session.user == "Guest":
+		return created_courses
+
+	CourseInstructor = frappe.qb.DocType("Course Instructor")
+	Course = frappe.qb.DocType("LMS Course")
+
+	query = (
+		frappe.qb.from_(CourseInstructor)
+		.join(Course)
+		.on(CourseInstructor.parent == Course.name)
+		.select(Course.name)
+		.where(CourseInstructor.instructor == frappe.session.user)
+		.orderby(Course.published_on, order=frappe.qb.desc)
+		.limit(3)
+	)
+
+	results = query.run(as_dict=True)
+	courses = [row["name"] for row in results]
+
+	for course in courses:
+		course_details = get_course_details(course)
+		created_courses.append(course_details)
+
+	return created_courses
+
+
+@frappe.whitelist()
+def get_created_batches():
+	created_batches = []
+	if frappe.session.user == "Guest":
+		return created_batches
+
+	CourseInstructor = frappe.qb.DocType("Course Instructor")
+	Batch = frappe.qb.DocType("LMS Batch")
+
+	query = (
+		frappe.qb.from_(CourseInstructor)
+		.join(Batch)
+		.on(CourseInstructor.parent == Batch.name)
+		.select(Batch.name)
+		.where(CourseInstructor.instructor == frappe.session.user)
+		.where(Batch.start_date >= getdate())
+		.orderby(Batch.start_date, order=frappe.qb.asc)
+		.limit(4)
+	)
+
+	results = query.run(as_dict=True)
+	batches = [row["name"] for row in results]
+
+	for batch in batches:
+		batch_details = get_batch_details(batch)
+		created_batches.append(batch_details)
+
+	return created_batches
+
+
+@frappe.whitelist()
+def get_admin_live_classes():
+	if frappe.session.user == "Guest":
+		return []
+
+	CourseInstructor = frappe.qb.DocType("Course Instructor")
+	LMSLiveClass = frappe.qb.DocType("LMS Live Class")
+
+	query = (
+		frappe.qb.from_(CourseInstructor)
+		.join(LMSLiveClass)
+		.on(CourseInstructor.parent == LMSLiveClass.batch_name)
+		.select(
+			LMSLiveClass.name,
+			LMSLiveClass.title,
+			LMSLiveClass.description,
+			LMSLiveClass.time,
+			LMSLiveClass.date,
+			LMSLiveClass.duration,
+			LMSLiveClass.attendees,
+			LMSLiveClass.start_url,
+			LMSLiveClass.join_url,
+			LMSLiveClass.owner,
+		)
+		.where(CourseInstructor.instructor == frappe.session.user)
+		.where(LMSLiveClass.date >= getdate())
+		.orderby(LMSLiveClass.date, order=frappe.qb.asc)
+		.limit(4)
+	)
+	results = query.run(as_dict=True)
+	return results
+
+
+@frappe.whitelist()
+def get_admin_evals():
+	if frappe.session.user == "Guest":
+		return []
+
+	evals = frappe.get_all(
+		"LMS Certificate Request",
+		{
+			"evaluator": frappe.session.user,
+			"date": [">=", getdate()],
+		},
+		[
+			"name",
+			"date",
+			"start_time",
+			"course",
+			"evaluator",
+			"google_meet_link",
+			"member",
+			"member_name",
+		],
+		limit=4,
+		order_by="date asc",
+	)
+
+	for evaluation in evals:
+		evaluation.course_title = frappe.db.get_value("LMS Course", evaluation.course, "title")
+
+	return evals
+
+
+@frappe.whitelist()
+def get_my_courses():
+	my_courses = []
+	if frappe.session.user == "Guest":
+		return my_courses
+
+	courses = get_my_latest_courses()
+
+	if not len(courses):
+		courses = get_featured_home_courses()
+
+	if not len(courses):
+		courses = get_popular_courses()
+
+	for course in courses:
+		my_courses.append(get_course_details(course))
+
+	return my_courses
+
+
+def get_my_latest_courses():
+	return frappe.get_all(
+		"LMS Enrollment",
+		{
+			"member": frappe.session.user,
+		},
+		order_by="modified desc",
+		limit=3,
+		pluck="course",
+	)
+
+
+def get_featured_home_courses():
+	return frappe.get_all(
+		"LMS Course",
+		{"published": 1, "featured": 1},
+		order_by="published_on desc",
+		limit=3,
+		pluck="name",
+	)
+
+
+def get_popular_courses():
+	return frappe.get_all(
+		"LMS Course",
+		{
+			"published": 1,
+		},
+		order_by="enrollments desc",
+		limit=3,
+		pluck="name",
+	)
+
+
+@frappe.whitelist()
+def get_my_batches():
+	my_batches = []
+	if frappe.session.user == "Guest":
+		return my_batches
+
+	batches = get_my_latest_batches()
+
+	if not len(batches):
+		batches = get_upcoming_batches()
+
+	for batch in batches:
+		batch_details = get_batch_details(batch)
+		if batch_details:
+			my_batches.append(batch_details)
+
+	return my_batches
+
+
+def get_my_latest_batches():
+	return frappe.get_all(
+		"LMS Batch Enrollment",
+		{
+			"member": frappe.session.user,
+		},
+		order_by="creation desc",
+		limit=4,
+		pluck="batch",
+	)
+
+
+def get_upcoming_batches():
+	return frappe.get_all(
+		"LMS Batch",
+		{
+			"published": 1,
+			"start_date": [">=", getdate()],
+		},
+		order_by="start_date asc",
+		limit=4,
+		pluck="name",
+	)
diff --git a/lms/lms/doctype/exercise_latest_submission/__init__.py b/lms/lms/doctype/exercise_latest_submission/__init__.py
deleted file mode 100644
index e69de29b..00000000
diff --git a/lms/lms/doctype/exercise_latest_submission/exercise_latest_submission.js b/lms/lms/doctype/exercise_latest_submission/exercise_latest_submission.js
deleted file mode 100644
index fd2d65e8..00000000
--- a/lms/lms/doctype/exercise_latest_submission/exercise_latest_submission.js
+++ /dev/null
@@ -1,7 +0,0 @@
-// Copyright (c) 2021, Frappe and contributors
-// For license information, please see license.txt
-
-frappe.ui.form.on("Exercise Latest Submission", {
-	// refresh: function(frm) {
-	// }
-});
diff --git a/lms/lms/doctype/exercise_latest_submission/exercise_latest_submission.json b/lms/lms/doctype/exercise_latest_submission/exercise_latest_submission.json
deleted file mode 100644
index 375f9758..00000000
--- a/lms/lms/doctype/exercise_latest_submission/exercise_latest_submission.json
+++ /dev/null
@@ -1,166 +0,0 @@
-{
- "actions": [],
- "creation": "2021-12-08 17:56:26.049675",
- "doctype": "DocType",
- "editable_grid": 1,
- "engine": "InnoDB",
- "field_order": [
-  "exercise",
-  "status",
-  "batch_old",
-  "column_break_4",
-  "exercise_title",
-  "course",
-  "lesson",
-  "section_break_8",
-  "solution",
-  "image",
-  "test_results",
-  "comments",
-  "latest_submission",
-  "member",
-  "member_email",
-  "member_cohort",
-  "member_subgroup"
- ],
- "fields": [
-  {
-   "fieldname": "exercise",
-   "fieldtype": "Link",
-   "in_list_view": 1,
-   "in_standard_filter": 1,
-   "label": "Exercise",
-   "options": "LMS Exercise",
-   "search_index": 1
-  },
-  {
-   "fieldname": "status",
-   "fieldtype": "Select",
-   "label": "Status",
-   "options": "Correct\nIncorrect"
-  },
-  {
-   "fieldname": "batch_old",
-   "fieldtype": "Link",
-   "label": "Batch Old",
-   "options": "LMS Batch Old"
-  },
-  {
-   "fieldname": "column_break_4",
-   "fieldtype": "Column Break"
-  },
-  {
-   "fetch_from": "exercise.title",
-   "fieldname": "exercise_title",
-   "fieldtype": "Data",
-   "label": "Exercise Title",
-   "read_only": 1
-  },
-  {
-   "fetch_from": "exercise.course",
-   "fieldname": "course",
-   "fieldtype": "Link",
-   "in_standard_filter": 1,
-   "label": "Course",
-   "options": "LMS Course",
-   "read_only": 1
-  },
-  {
-   "fetch_from": "exercise.lesson",
-   "fieldname": "lesson",
-   "fieldtype": "Link",
-   "label": "Lesson",
-   "options": "Course Lesson"
-  },
-  {
-   "fieldname": "section_break_8",
-   "fieldtype": "Section Break"
-  },
-  {
-   "fetch_from": "latest_submission.solution",
-   "fieldname": "solution",
-   "fieldtype": "Code",
-   "label": "Solution"
-  },
-  {
-   "fetch_from": "latest_submission.image",
-   "fieldname": "image",
-   "fieldtype": "Code",
-   "label": "Image",
-   "read_only": 1
-  },
-  {
-   "fetch_from": "latest_submission.test_results",
-   "fieldname": "test_results",
-   "fieldtype": "Small Text",
-   "label": "Test Results"
-  },
-  {
-   "fieldname": "comments",
-   "fieldtype": "Small Text",
-   "label": "Comments"
-  },
-  {
-   "fieldname": "latest_submission",
-   "fieldtype": "Link",
-   "label": "Latest Submission",
-   "options": "Exercise Submission"
-  },
-  {
-   "fieldname": "member",
-   "fieldtype": "Link",
-   "label": "Member",
-   "options": "LMS Enrollment"
-  },
-  {
-   "fetch_from": "member.member",
-   "fieldname": "member_email",
-   "fieldtype": "Link",
-   "in_list_view": 1,
-   "in_standard_filter": 1,
-   "label": "Member Email",
-   "options": "User",
-   "search_index": 1
-  },
-  {
-   "fetch_from": "member.cohort",
-   "fieldname": "member_cohort",
-   "fieldtype": "Link",
-   "label": "Member Cohort",
-   "options": "Cohort",
-   "search_index": 1
-  },
-  {
-   "fetch_from": "member.subgroup",
-   "fieldname": "member_subgroup",
-   "fieldtype": "Link",
-   "label": "Member Subgroup",
-   "options": "Cohort Subgroup",
-   "search_index": 1
-  }
- ],
- "index_web_pages_for_search": 1,
- "links": [],
- "modified": "2021-12-08 22:58:46.312863",
- "modified_by": "Administrator",
- "module": "LMS",
- "name": "Exercise Latest Submission",
- "owner": "Administrator",
- "permissions": [
-  {
-   "create": 1,
-   "delete": 1,
-   "email": 1,
-   "export": 1,
-   "print": 1,
-   "read": 1,
-   "report": 1,
-   "role": "System Manager",
-   "share": 1,
-   "write": 1
-  }
- ],
- "sort_field": "modified",
- "sort_order": "DESC",
- "track_changes": 1
-}
diff --git a/lms/lms/doctype/exercise_latest_submission/exercise_latest_submission.py b/lms/lms/doctype/exercise_latest_submission/exercise_latest_submission.py
deleted file mode 100644
index 11a1cf39..00000000
--- a/lms/lms/doctype/exercise_latest_submission/exercise_latest_submission.py
+++ /dev/null
@@ -1,9 +0,0 @@
-# Copyright (c) 2021, Frappe and contributors
-# For license information, please see license.txt
-
-# import frappe
-from frappe.model.document import Document
-
-
-class ExerciseLatestSubmission(Document):
-	pass
diff --git a/lms/lms/doctype/exercise_latest_submission/test_exercise_latest_submission.py b/lms/lms/doctype/exercise_latest_submission/test_exercise_latest_submission.py
deleted file mode 100644
index 7f8809c6..00000000
--- a/lms/lms/doctype/exercise_latest_submission/test_exercise_latest_submission.py
+++ /dev/null
@@ -1,9 +0,0 @@
-# Copyright (c) 2021, Frappe and Contributors
-# See license.txt
-
-# import frappe
-import unittest
-
-
-class TestExerciseLatestSubmission(unittest.TestCase):
-	pass
diff --git a/lms/lms/doctype/exercise_submission/__init__.py b/lms/lms/doctype/exercise_submission/__init__.py
deleted file mode 100644
index e69de29b..00000000
diff --git a/lms/lms/doctype/exercise_submission/exercise_submission.js b/lms/lms/doctype/exercise_submission/exercise_submission.js
deleted file mode 100644
index fbbea522..00000000
--- a/lms/lms/doctype/exercise_submission/exercise_submission.js
+++ /dev/null
@@ -1,7 +0,0 @@
-// Copyright (c) 2021, FOSS United and contributors
-// For license information, please see license.txt
-
-frappe.ui.form.on("Exercise Submission", {
-	// refresh: function(frm) {
-	// }
-});
diff --git a/lms/lms/doctype/exercise_submission/exercise_submission.json b/lms/lms/doctype/exercise_submission/exercise_submission.json
deleted file mode 100644
index 9b5a6fa0..00000000
--- a/lms/lms/doctype/exercise_submission/exercise_submission.json
+++ /dev/null
@@ -1,126 +0,0 @@
-{
- "actions": [],
- "creation": "2021-05-19 11:41:18.108316",
- "doctype": "DocType",
- "editable_grid": 1,
- "engine": "InnoDB",
- "field_order": [
-  "exercise",
-  "status",
-  "batch_old",
-  "column_break_4",
-  "exercise_title",
-  "course",
-  "lesson",
-  "section_break_8",
-  "solution",
-  "image",
-  "test_results",
-  "comments",
-  "member"
- ],
- "fields": [
-  {
-   "fieldname": "exercise",
-   "fieldtype": "Link",
-   "in_list_view": 1,
-   "label": "Exercise",
-   "options": "LMS Exercise"
-  },
-  {
-   "fetch_from": "exercise.title",
-   "fieldname": "exercise_title",
-   "fieldtype": "Data",
-   "in_list_view": 1,
-   "label": "Exercise Title",
-   "read_only": 1
-  },
-  {
-   "fetch_from": "exercise.course",
-   "fieldname": "course",
-   "fieldtype": "Link",
-   "label": "Course",
-   "options": "LMS Course",
-   "read_only": 1
-  },
-  {
-   "fieldname": "batch_old",
-   "fieldtype": "Link",
-   "label": "Batch Old",
-   "options": "LMS Batch Old"
-  },
-  {
-   "fetch_from": "exercise.lesson",
-   "fieldname": "lesson",
-   "fieldtype": "Link",
-   "label": "Lesson",
-   "options": "Course Lesson"
-  },
-  {
-   "fieldname": "image",
-   "fieldtype": "Code",
-   "label": "Image",
-   "read_only": 1
-  },
-  {
-   "fieldname": "status",
-   "fieldtype": "Select",
-   "label": "Status",
-   "options": "Correct\nIncorrect"
-  },
-  {
-   "fieldname": "test_results",
-   "fieldtype": "Small Text",
-   "label": "Test Results"
-  },
-  {
-   "fieldname": "comments",
-   "fieldtype": "Small Text",
-   "label": "Comments"
-  },
-  {
-   "fieldname": "solution",
-   "fieldtype": "Code",
-   "in_list_view": 1,
-   "label": "Solution"
-  },
-  {
-   "fieldname": "column_break_4",
-   "fieldtype": "Column Break"
-  },
-  {
-   "fieldname": "section_break_8",
-   "fieldtype": "Section Break"
-  },
-  {
-   "fieldname": "member",
-   "fieldtype": "Link",
-   "label": "Member",
-   "options": "LMS Enrollment"
-  }
- ],
- "index_web_pages_for_search": 1,
- "links": [],
- "modified": "2021-12-08 22:25:05.809377",
- "modified_by": "Administrator",
- "module": "LMS",
- "name": "Exercise Submission",
- "owner": "Administrator",
- "permissions": [
-  {
-   "create": 1,
-   "delete": 1,
-   "email": 1,
-   "export": 1,
-   "print": 1,
-   "read": 1,
-   "report": 1,
-   "role": "System Manager",
-   "share": 1,
-   "write": 1
-  }
- ],
- "sort_field": "modified",
- "sort_order": "DESC",
- "track_changes": 1
-}
diff --git a/lms/lms/doctype/exercise_submission/exercise_submission.py b/lms/lms/doctype/exercise_submission/exercise_submission.py
deleted file mode 100644
index cf2c80cc..00000000
--- a/lms/lms/doctype/exercise_submission/exercise_submission.py
+++ /dev/null
@@ -1,29 +0,0 @@
-# Copyright (c) 2021, FOSS United and contributors
-# For license information, please see license.txt
-
-import frappe
-from frappe.model.document import Document
-
-
-class ExerciseSubmission(Document):
-	def on_update(self):
-		self.update_latest_submission()
-
-	def update_latest_submission(self):
-		names = frappe.get_all(
-			"Exercise Latest Submission", {"exercise": self.exercise, "member": self.member}
-		)
-		if names:
-			doc = frappe.get_doc("Exercise Latest Submission", names[0])
-			doc.latest_submission = self.name
-			doc.save(ignore_permissions=True)
-		else:
-			doc = frappe.get_doc(
-				{
-					"doctype": "Exercise Latest Submission",
-					"exercise": self.exercise,
-					"member": self.member,
-					"latest_submission": self.name,
-				}
-			)
-			doc.insert(ignore_permissions=True)
diff --git a/lms/lms/doctype/exercise_submission/test_exercise_submission.py b/lms/lms/doctype/exercise_submission/test_exercise_submission.py
deleted file mode 100644
index 71704011..00000000
--- a/lms/lms/doctype/exercise_submission/test_exercise_submission.py
+++ /dev/null
@@ -1,9 +0,0 @@
-# Copyright (c) 2021, FOSS United and Contributors
-# See license.txt
-
-# import frappe
-import unittest
-
-
-class TestExerciseSubmission(unittest.TestCase):
-	pass
diff --git a/lms/lms/doctype/lms_batch_old/__init__.py b/lms/lms/doctype/lms_batch_old/__init__.py
deleted file mode 100644
index e69de29b..00000000
diff --git a/lms/lms/doctype/lms_batch_old/lms_batch_old.js b/lms/lms/doctype/lms_batch_old/lms_batch_old.js
deleted file mode 100644
index 8c9c7d89..00000000
--- a/lms/lms/doctype/lms_batch_old/lms_batch_old.js
+++ /dev/null
@@ -1,7 +0,0 @@
-// Copyright (c) 2021, FOSS United and contributors
-// For license information, please see license.txt
-
-frappe.ui.form.on("LMS Batch Old", {
-	// refresh: function(frm) {
-	// }
-});
diff --git a/lms/lms/doctype/lms_batch_old/lms_batch_old.json b/lms/lms/doctype/lms_batch_old/lms_batch_old.json
deleted file mode 100644
index 5f643ca2..00000000
--- a/lms/lms/doctype/lms_batch_old/lms_batch_old.json
+++ /dev/null
@@ -1,150 +0,0 @@
-{
- "actions": [],
- "autoname": "format: BATCH-{#####}",
- "creation": "2021-03-18 19:37:34.614796",
- "doctype": "DocType",
- "editable_grid": 1,
- "engine": "InnoDB",
- "field_order": [
-  "course",
-  "start_date",
-  "start_time",
-  "column_break_3",
-  "title",
-  "sessions_on",
-  "end_time",
-  "section_break_5",
-  "description",
-  "section_break_7",
-  "visibility",
-  "membership",
-  "column_break_9",
-  "status",
-  "stage"
- ],
- "fields": [
-  {
-   "fieldname": "course",
-   "fieldtype": "Link",
-   "in_list_view": 1,
-   "in_standard_filter": 1,
-   "label": "Course",
-   "options": "LMS Course",
-   "reqd": 1
-  },
-  {
-   "fieldname": "title",
-   "fieldtype": "Data",
-   "label": "Title",
-   "reqd": 1
-  },
-  {
-   "fieldname": "description",
-   "fieldtype": "Markdown Editor",
-   "label": "Description"
-  },
-  {
-   "default": "Public",
-   "fieldname": "visibility",
-   "fieldtype": "Select",
-   "in_list_view": 1,
-   "label": "Visibility",
-   "options": "Public\nUnlisted\nPrivate"
-  },
-  {
-   "fieldname": "membership",
-   "fieldtype": "Select",
-   "label": "Membership",
-   "options": "\nOpen\nRestricted\nInvite Only\nClosed"
-  },
-  {
-   "default": "Active",
-   "fieldname": "status",
-   "fieldtype": "Select",
-   "in_list_view": 1,
-   "label": "Status",
-   "options": "Active\nInactive"
-  },
-  {
-   "default": "Ready",
-   "fieldname": "stage",
-   "fieldtype": "Select",
-   "label": "Stage",
-   "options": "Ready\nIn Progress\nCompleted\nCancelled"
-  },
-  {
-   "fieldname": "column_break_3",
-   "fieldtype": "Column Break"
-  },
-  {
-   "fieldname": "section_break_5",
-   "fieldtype": "Section Break",
-   "label": "Batch Description"
-  },
-  {
-   "fieldname": "column_break_9",
-   "fieldtype": "Column Break"
-  },
-  {
-   "fieldname": "section_break_7",
-   "fieldtype": "Section Break",
-   "label": "Batch Settings"
-  },
-  {
-   "fieldname": "start_date",
-   "fieldtype": "Date",
-   "in_list_view": 1,
-   "label": "Start Date"
-  },
-  {
-   "fieldname": "start_time",
-   "fieldtype": "Time",
-   "in_list_view": 1,
-   "label": "Start Time"
-  },
-  {
-   "fieldname": "sessions_on",
-   "fieldtype": "Data",
-   "label": "Sessions On Days"
-  },
-  {
-   "fieldname": "end_time",
-   "fieldtype": "Time",
-   "in_list_view": 1,
-   "label": "End Time"
-  }
- ],
- "index_web_pages_for_search": 1,
- "links": [
-  {
-   "group": "Members",
-   "link_doctype": "LMS Enrollment",
-   "link_fieldname": "batch_old"
-  }
- ],
- "modified": "2022-09-28 18:43:22.955907",
- "modified_by": "Administrator",
- "module": "LMS",
- "name": "LMS Batch Old",
- "naming_rule": "Expression",
- "owner": "Administrator",
- "permissions": [
-  {
-   "create": 1,
-   "delete": 1,
-   "email": 1,
-   "export": 1,
-   "print": 1,
-   "read": 1,
-   "report": 1,
-   "role": "System Manager",
-   "share": 1,
-   "write": 1
-  }
- ],
- "quick_entry": 1,
- "sort_field": "modified",
- "sort_order": "DESC",
- "states": [],
- "track_changes": 1
-}
\ No newline at end of file
diff --git a/lms/lms/doctype/lms_batch_old/lms_batch_old.py b/lms/lms/doctype/lms_batch_old/lms_batch_old.py
deleted file mode 100644
index a7a4aaac..00000000
--- a/lms/lms/doctype/lms_batch_old/lms_batch_old.py
+++ /dev/null
@@ -1,90 +0,0 @@
-# Copyright (c) 2021, FOSS United and contributors
-# For license information, please see license.txt
-
-import frappe
-from frappe import _
-from frappe.model.document import Document
-
-from lms.lms.doctype.lms_enrollment.lms_enrollment import create_membership
-from lms.lms.utils import is_mentor
-
-
-class LMSBatchOld(Document):
-	def validate(self):
-		pass
-		# self.validate_if_mentor()
-
-	def validate_if_mentor(self):
-		if not is_mentor(self.course, frappe.session.user):
-			course_title = frappe.db.get_value("LMS Course", self.course, "title")
-			frappe.throw(_("You are not a mentor of the course {0}").format(course_title))
-
-	def after_insert(self):
-		create_membership(batch=self.name, course=self.course, member_type="Mentor")
-
-	def is_member(self, email, member_type=None):
-		"""Checks if a person is part of a batch.
-
-		If member_type is specified, checks if the person is a Student/Mentor.
-		"""
-
-		filters = {"batch_old": self.name, "member": email}
-		if member_type:
-			filters["member_type"] = member_type
-		return frappe.db.exists("LMS Enrollment", filters)
-
-	def get_membership(self, email):
-		"""Returns the membership document of given user."""
-		name = frappe.get_value(
-			doctype="LMS Enrollment",
-			filters={"batch_old": self.name, "member": email},
-			fieldname="name",
-		)
-		return frappe.get_doc("LMS Enrollment", name)
-
-	def get_current_lesson(self, user):
-		"""Returns the name of the current lesson for the given user."""
-		membership = self.get_membership(user)
-		return membership and membership.current_lesson
-
-
-@frappe.whitelist()
-def save_message(message, batch):
-	doc = frappe.get_doc(
-		{
-			"doctype": "LMS Message",
-			"batch_old": batch,
-			"author": frappe.session.user,
-			"message": message,
-		}
-	)
-	doc.save(ignore_permissions=True)
-
-
-def switch_batch(course_name, email, batch_name):
-	"""Switches the user from the current batch of the course to a new batch."""
-	membership = frappe.get_last_doc("LMS Enrollment", filters={"course": course_name, "member": email})
-
-	batch = frappe.get_doc("LMS Batch Old", batch_name)
-	if not batch:
-		raise ValueError(f"Invalid Batch: {batch_name}")
-
-	if batch.course != course_name:
-		raise ValueError("Can not switch batches across courses")
-
-	if batch.is_member(email):
-		print(f"{email} is already a member of {batch.title}")
-		return
-
-	old_batch = frappe.get_doc("LMS Batch Old", membership.batch_old)
-
-	membership.batch_old = batch_name
-	membership.save()
-
-	# update exercise submissions
-	filters = {"owner": email, "batch_old": old_batch.name}
-	for name in frappe.db.get_all("Exercise Submission", filters=filters, pluck="name"):
-		doc = frappe.get_doc("Exercise Submission", name)
-		print("updating exercise submission", name)
-		doc.batch_old = batch_name
-		doc.save()
diff --git a/lms/lms/doctype/lms_batch_old/test_lms_batch_old.py b/lms/lms/doctype/lms_batch_old/test_lms_batch_old.py
deleted file mode 100644
index e3dd1283..00000000
--- a/lms/lms/doctype/lms_batch_old/test_lms_batch_old.py
+++ /dev/null
@@ -1,9 +0,0 @@
-# Copyright (c) 2021, FOSS United and Contributors
-# See license.txt
-
-# import frappe
-import unittest
-
-
-class TestLMSBatchOld(unittest.TestCase):
-	pass
diff --git a/lms/lms/doctype/lms_certificate/lms_certificate.py b/lms/lms/doctype/lms_certificate/lms_certificate.py
index a2f18abb..d67f9b7d 100644
--- a/lms/lms/doctype/lms_certificate/lms_certificate.py
+++ b/lms/lms/doctype/lms_certificate/lms_certificate.py
@@ -6,9 +6,7 @@ from frappe import _
 from frappe.email.doctype.email_template.email_template import get_email_template
 from frappe.model.document import Document
 from frappe.model.naming import make_autoname
-from frappe.utils import add_years, nowdate
-
-from lms.lms.utils import is_certified
+from frappe.utils import nowdate
 
 
 class LMSCertificate(Document):
@@ -113,6 +111,13 @@ def has_website_permission(doc, ptype, user, verbose=False):
 	return False
 
 
+def is_certified(course):
+	certificate = frappe.get_all("LMS Certificate", {"member": frappe.session.user, "course": course})
+	if len(certificate):
+		return certificate[0].name
+	return
+
+
 @frappe.whitelist()
 def create_certificate(course):
 	if is_certified(course):
diff --git a/lms/lms/doctype/lms_course/lms_course.py b/lms/lms/doctype/lms_course/lms_course.py
index 85e6dbdf..5f8acb55 100644
--- a/lms/lms/doctype/lms_course/lms_course.py
+++ b/lms/lms/doctype/lms_course/lms_course.py
@@ -156,29 +156,6 @@ class LMSCourse(Document):
 		doc = frappe.get_doc({"doctype": "LMS Course Mentor Mapping", "course": self.name, "mentor": email})
 		doc.insert()
 
-	def get_student_batch(self, email):
-		"""Returns the batch the given student is part of.
-
-		Returns None if the student is not part of any batch.
-		"""
-		if not email:
-			return
-
-		batch_name = frappe.get_value(
-			doctype="LMS Enrollment",
-			filters={"course": self.name, "member_type": "Student", "member": email},
-			fieldname="batch_old",
-		)
-		return batch_name and frappe.get_doc("LMS Batch Old", batch_name)
-
-	def get_batches(self, mentor=None):
-		batches = frappe.get_all("LMS Batch Old", {"course": self.name})
-		if mentor:
-			# TODO: optimize this
-			memberships = frappe.db.get_all("LMS Enrollment", {"member": mentor}, ["batch_old"])
-			batch_names = {m.batch_old for m in memberships}
-			return [b for b in batches if b.name in batch_names]
-
 	def get_cohorts(self):
 		return frappe.get_all(
 			"Cohort",
@@ -204,14 +181,6 @@ class LMSCourse(Document):
 				exercise.save()
 				i += 1
 
-	def get_all_memberships(self, member):
-		all_memberships = frappe.get_all(
-			"LMS Enrollment", {"member": member, "course": self.name}, ["batch_old"]
-		)
-		for membership in all_memberships:
-			membership.batch_title = frappe.db.get_value("LMS Batch Old", membership.batch_old, "title")
-		return all_memberships
-
 
 @frappe.whitelist()
 def reindex_exercises(doc):
diff --git a/lms/lms/doctype/lms_course/test_lms_course.py b/lms/lms/doctype/lms_course/test_lms_course.py
index d3862e6b..72d7dadc 100644
--- a/lms/lms/doctype/lms_course/test_lms_course.py
+++ b/lms/lms/doctype/lms_course/test_lms_course.py
@@ -13,18 +13,6 @@ class TestLMSCourse(unittest.TestCase):
 		course = new_course("Test Course")
 		assert course.title == "Test Course"
 
-	# disabled this test as it is failing
-	def _test_add_mentors(self):
-		course = new_course("Test Course")
-		assert course.get_mentors() == []
-
-		new_user("Tester", "tester@example.com")
-		course.add_mentor("tester@example.com")
-
-		mentors = course.get_mentors()
-		mentors_data = [dict(email=mentor.email, batch_count=mentor.batch_count) for mentor in mentors]
-		assert mentors_data == [{"email": "tester@example.com", "batch_count": 0}]
-
 	def tearDown(self):
 		if frappe.db.exists("User", "tester@example.com"):
 			frappe.delete_doc("User", "tester@example.com")
@@ -37,7 +25,6 @@ class TestLMSCourse(unittest.TestCase):
 			frappe.db.delete("LMS Enrollment", {"course": "test-course"})
 			frappe.db.delete("Course Lesson", {"course": "test-course"})
 			frappe.db.delete("Course Chapter", {"course": "test-course"})
-			frappe.db.delete("LMS Batch Old", {"course": "test-course"})
 			frappe.db.delete("LMS Course Mentor Mapping", {"course": "test-course"})
 			frappe.db.delete("Course Instructor", {"parent": "test-course"})
 			frappe.db.sql("delete from `tabCourse Instructor`")
diff --git a/lms/lms/doctype/lms_course_review/lms_course_review.py b/lms/lms/doctype/lms_course_review/lms_course_review.py
index fc1befb3..8f754ad4 100644
--- a/lms/lms/doctype/lms_course_review/lms_course_review.py
+++ b/lms/lms/doctype/lms_course_review/lms_course_review.py
@@ -2,17 +2,24 @@
 # For license information, please see license.txt
 
 import frappe
+from frappe import _
 from frappe.model.document import Document
 from frappe.utils import cint
 
 
 class LMSCourseReview(Document):
 	def validate(self):
+		self.validate_enrollment()
 		self.validate_if_already_reviewed()
 
+	def validate_enrollment(self):
+		enrollment = frappe.db.exists("LMS Enrollment", {"course": self.course, "member": self.owner})
+		if not enrollment:
+			frappe.throw(_("You must be enrolled in the course to submit a review"))
+
 	def validate_if_already_reviewed(self):
 		if frappe.db.exists("LMS Course Review", {"course": self.course, "owner": self.owner}):
-			frappe.throw(frappe._("You have already reviewed this course"))
+			frappe.throw(_("You have already reviewed this course"))
 
 
 @frappe.whitelist()
diff --git a/lms/lms/doctype/lms_enrollment/lms_enrollment.json b/lms/lms/doctype/lms_enrollment/lms_enrollment.json
index cfcdf76b..887724f2 100644
--- a/lms/lms/doctype/lms_enrollment/lms_enrollment.json
+++ b/lms/lms/doctype/lms_enrollment/lms_enrollment.json
@@ -27,12 +27,6 @@
   "role"
  ],
  "fields": [
-  {
-   "fieldname": "batch_old",
-   "fieldtype": "Link",
-   "label": "Batch Old",
-   "options": "LMS Batch Old"
-  },
   {
    "fieldname": "member",
    "fieldtype": "Link",
@@ -155,7 +149,7 @@
  "grid_page_length": 50,
  "index_web_pages_for_search": 1,
  "links": [],
- "modified": "2025-07-02 21:27:30.733482",
+ "modified": "2025-12-08 21:27:30.733482",
  "modified_by": "Administrator",
  "module": "LMS",
  "name": "LMS Enrollment",
diff --git a/lms/lms/doctype/lms_enrollment/lms_enrollment.py b/lms/lms/doctype/lms_enrollment/lms_enrollment.py
index 9112c7a6..fbf6994f 100644
--- a/lms/lms/doctype/lms_enrollment/lms_enrollment.py
+++ b/lms/lms/doctype/lms_enrollment/lms_enrollment.py
@@ -10,7 +10,6 @@ from frappe.utils import ceil
 class LMSEnrollment(Document):
 	def validate(self):
 		self.validate_membership_in_same_batch()
-		self.validate_membership_in_different_batch_same_course()
 
 	def on_update(self):
 		update_program_progress(self.member)
@@ -32,33 +31,6 @@ class LMSEnrollment(Document):
 				)
 			)
 
-	def validate_membership_in_different_batch_same_course(self):
-		"""Ensures that a studnet is only part of one batch."""
-		# nothing to worry if the member is not a student
-		if self.member_type != "Student":
-			return
-
-		course = frappe.db.get_value("LMS Batch Old", self.batch_old, "course")
-		memberships = frappe.get_all(
-			"LMS Enrollment",
-			filters={
-				"member": self.member,
-				"name": ["!=", self.name],
-				"member_type": "Student",
-				"course": self.course,
-			},
-			fields=["batch_old", "member_type", "name"],
-		)
-
-		if memberships:
-			membership = memberships[0]
-			member_name = frappe.db.get_value("User", self.member, "full_name")
-			frappe.throw(
-				_("{0} is already a Student of {1} course through {2} batch").format(
-					member_name, course, membership.batch_old
-				)
-			)
-
 
 def update_program_progress(member):
 	programs = frappe.get_all("LMS Program Member", {"member": member}, ["parent", "name"])
diff --git a/lms/lms/doctype/lms_exercise/__init__.py b/lms/lms/doctype/lms_exercise/__init__.py
deleted file mode 100644
index e69de29b..00000000
diff --git a/lms/lms/doctype/lms_exercise/lms_exercise.js b/lms/lms/doctype/lms_exercise/lms_exercise.js
deleted file mode 100644
index d86d34c6..00000000
--- a/lms/lms/doctype/lms_exercise/lms_exercise.js
+++ /dev/null
@@ -1,7 +0,0 @@
-// Copyright (c) 2021, FOSS United and contributors
-// For license information, please see license.txt
-
-frappe.ui.form.on("LMS Exercise", {
-	// refresh: function(frm) {
-	// }
-});
diff --git a/lms/lms/doctype/lms_exercise/lms_exercise.json b/lms/lms/doctype/lms_exercise/lms_exercise.json
deleted file mode 100644
index 8c0068de..00000000
--- a/lms/lms/doctype/lms_exercise/lms_exercise.json
+++ /dev/null
@@ -1,123 +0,0 @@
-{
- "actions": [],
- "allow_rename": 1,
- "creation": "2021-05-19 17:43:39.923430",
- "doctype": "DocType",
- "editable_grid": 1,
- "engine": "InnoDB",
- "field_order": [
-  "title",
-  "description",
-  "code",
-  "answer",
-  "column_break_4",
-  "course",
-  "hints",
-  "tests",
-  "image",
-  "lesson",
-  "index_",
-  "index_label"
- ],
- "fields": [
-  {
-   "fieldname": "title",
-   "fieldtype": "Data",
-   "in_list_view": 1,
-   "label": "Title"
-  },
-  {
-   "fieldname": "course",
-   "fieldtype": "Link",
-   "in_list_view": 1,
-   "label": "Course",
-   "options": "LMS Course"
-  },
-  {
-   "columns": 4,
-   "fieldname": "description",
-   "fieldtype": "Small Text",
-   "in_list_view": 1,
-   "label": "Description"
-  },
-  {
-   "columns": 4,
-   "fieldname": "answer",
-   "fieldtype": "Code",
-   "label": "Answer"
-  },
-  {
-   "fieldname": "tests",
-   "fieldtype": "Code",
-   "label": "Tests"
-  },
-  {
-   "fieldname": "column_break_4",
-   "fieldtype": "Column Break"
-  },
-  {
-   "columns": 4,
-   "fieldname": "hints",
-   "fieldtype": "Small Text",
-   "label": "Hints"
-  },
-  {
-   "columns": 4,
-   "fieldname": "code",
-   "fieldtype": "Code",
-   "label": "Code"
-  },
-  {
-   "fieldname": "image",
-   "fieldtype": "Code",
-   "label": "Image",
-   "read_only": 1
-  },
-  {
-   "fieldname": "lesson",
-   "fieldtype": "Link",
-   "in_list_view": 1,
-   "label": "Lesson",
-   "options": "Course Lesson"
-  },
-  {
-   "fieldname": "index_",
-   "fieldtype": "Int",
-   "label": "Index",
-   "read_only": 1
-  },
-  {
-   "fieldname": "index_label",
-   "fieldtype": "Data",
-   "in_list_view": 1,
-   "label": "Index Label",
-   "read_only": 1
-  }
- ],
- "index_web_pages_for_search": 1,
- "links": [],
- "modified": "2021-09-29 15:27:55.585874",
- "modified_by": "Administrator",
- "module": "LMS",
- "name": "LMS Exercise",
- "owner": "Administrator",
- "permissions": [
-  {
-   "create": 1,
-   "delete": 1,
-   "email": 1,
-   "export": 1,
-   "print": 1,
-   "read": 1,
-   "report": 1,
-   "role": "System Manager",
-   "share": 1,
-   "write": 1
-  }
- ],
- "search_fields": "title",
- "sort_field": "index_label",
- "sort_order": "ASC",
- "title_field": "title",
- "track_changes": 1
-}
diff --git a/lms/lms/doctype/lms_exercise/lms_exercise.py b/lms/lms/doctype/lms_exercise/lms_exercise.py
deleted file mode 100644
index e43d5a85..00000000
--- a/lms/lms/doctype/lms_exercise/lms_exercise.py
+++ /dev/null
@@ -1,52 +0,0 @@
-# Copyright (c) 2021, FOSS United and contributors
-# For license information, please see license.txt
-
-import frappe
-from frappe.model.document import Document
-
-from lms.lms.utils import get_membership
-
-
-class LMSExercise(Document):
-	def get_user_submission(self):
-		"""Returns the latest submission for this user."""
-		user = frappe.session.user
-		if not user or user == "Guest":
-			return
-
-		result = frappe.get_all(
-			"Exercise Submission",
-			fields="*",
-			filters={"owner": user, "exercise": self.name},
-			order_by="creation desc",
-			page_length=1,
-		)
-
-		if result:
-			return result[0]
-
-	def submit(self, code):
-		"""Submits the given code as solution to exercise."""
-		user = frappe.session.user
-		if not user or user == "Guest":
-			return
-
-		old_submission = self.get_user_submission()
-		if old_submission and old_submission.solution == code:
-			return old_submission
-
-		member = get_membership(self.course, frappe.session.user)
-
-		doc = frappe.get_doc(
-			doctype="Exercise Submission",
-			exercise=self.name,
-			exercise_title=self.title,
-			course=self.course,
-			lesson=self.lesson,
-			batch=member.batch_old,
-			solution=code,
-			member=member.name,
-		)
-		doc.insert(ignore_permissions=True)
-
-		return doc
diff --git a/lms/lms/doctype/lms_exercise/test_lms_exercise.py b/lms/lms/doctype/lms_exercise/test_lms_exercise.py
deleted file mode 100644
index 76ede444..00000000
--- a/lms/lms/doctype/lms_exercise/test_lms_exercise.py
+++ /dev/null
@@ -1,10 +0,0 @@
-# Copyright (c) 2021, FOSS United and Contributors
-# See license.txt
-
-import unittest
-
-# import frappe
-
-
-class TestLMSExercise(unittest.TestCase):
-	pass
diff --git a/lms/lms/test_utils.py b/lms/lms/test_utils.py
index 34347b3a..ecec9f3e 100644
--- a/lms/lms/test_utils.py
+++ b/lms/lms/test_utils.py
@@ -2,16 +2,169 @@ import unittest
 
 import frappe
 
-from .utils import slugify
+from .utils import (
+	get_average_rating,
+	get_chapters,
+	get_instructors,
+	get_lessons,
+	get_membership,
+	get_reviews,
+	get_tags,
+	slugify,
+)
 
 
 class TestUtils(unittest.TestCase):
-	def test_simple(self):
+	def setUp(self):
+		self.create_a_course()
+		self.add_chapters()
+		self.add_lessons()
+
+		self.student1 = self.create_student("student1@example.com", "Ashley", "Smith")
+		self.student2 = self.create_student("student2@example.com", "John", "Doe")
+
+		self.add_enrollment(self.course.name, self.student1.email)
+		self.add_enrollment(self.course.name, self.student2.email)
+
+		self.add_rating(self.course.name, self.student1.email, 0.8, "Good course")
+		self.add_rating(self.course.name, self.student2.email, 1, "Excellent course")
+
+	def create_a_course(self):
+		course = frappe.new_doc("LMS Course")
+		course.title = "Utility Course"
+		course.short_introduction = "A course to test utilities of Frappe Learning"
+		course.description = "This is a detailed description of the Utility Course."
+		course.tags = "Frappe,Learning,Utility"
+		course.published = 1
+		course.append("instructors", {"instructor": "frappe@example.com"})
+		course.save()
+		self.course = course
+
+	def add_chapters(self):
+		chapters = []
+		for i in range(1, 4):
+			chapter = frappe.new_doc("Course Chapter")
+			chapter.course = self.course.name
+			chapter.title = f"Chapter {i}"
+			chapter.save()
+			chapters.append(chapter)
+
+		self.course.reload()
+		for chapter in chapters:
+			self.course.append("chapters", {"chapter": chapter.name})
+
+		self.course.save()
+
+	def add_lessons(self):
+		for chapter in self.course.chapters:
+			chapterDoc = frappe.get_doc("Course Chapter", chapter.chapter)
+			lessons = []
+			for j in range(1, 3):
+				lesson = frappe.new_doc("Course Lesson")
+				lesson.course = self.course.name
+				lesson.chapter = chapter.chapter
+				lesson.title = f"Lesson {j} of {chapter.chapter}"
+				content = '{"time":1765194986690,"blocks":[{"id":"dkLzbW14ds","type":"markdown","data":{"text":"This is a simple content for the current lesson."}},{"id":"KBwuWPc8rV","type":"markdown","data":{"text":""}}],"version":"2.29.0"}'
+				lesson.content = content
+				lesson.save()
+				lessons.append(lesson)
+
+			for lesson in lessons:
+				chapterDoc.append("lessons", {"lesson": lesson.name})
+			chapterDoc.save()
+
+	def create_student(self, email, first_name, last_name):
+		student = frappe.new_doc("User")
+		student.email = email
+		student.first_name = first_name
+		student.last_name = last_name
+		student.user_type = "Website User"
+		student.append("roles", {"role": "LMS Student"})
+		student.save()
+		return student
+
+	def test_simple_slugs(self):
 		self.assertEqual(slugify("hello-world"), "hello-world")
 		self.assertEqual(slugify("Hello World"), "hello-world")
 		self.assertEqual(slugify("Hello, World!"), "hello-world")
 
-	def test_duplicates(self):
+	def test_duplicates_slugs(self):
 		self.assertEqual(slugify("Hello World", ["hello-world"]), "hello-world-2")
-
 		self.assertEqual(slugify("Hello World", ["hello-world", "hello-world-2"]), "hello-world-3")
+
+	def add_enrollment(self, course, member):
+		enrollment = frappe.new_doc("LMS Enrollment")
+		enrollment.course = course
+		enrollment.member = member
+		enrollment.save()
+
+	def test_get_membership(self):
+		membership = get_membership(self.course.name, self.student1.email)
+		self.assertIsNotNone(membership)
+		self.assertEqual(membership.course, self.course.name)
+		self.assertEqual(membership.member, self.student1.email)
+
+	def test_get_chapters(self):
+		chapters = get_chapters(self.course.name)
+		self.assertEqual(len(chapters), len(self.course.chapters))
+
+		for i, chapter in enumerate(chapters, start=1):
+			self.assertEqual(chapter.title, f"Chapter {i}")
+
+	def test_get_lessons(self):
+		lessons = get_lessons(self.course.name)
+		all_lessons = frappe.db.count("Course Lesson", {"course": self.course.name})
+		self.assertEqual(len(lessons), all_lessons)
+
+		for chapter in self.course.chapters:
+			chapter_lessons = [lesson for lesson in lessons if lesson.chapter == chapter.chapter]
+			self.assertEqual(len(chapter_lessons), 2)
+			for j, lesson in enumerate(chapter_lessons, start=1):
+				self.assertEqual(lesson.title, f"Lesson {j} of {chapter.chapter}")
+				self.assertEqual(lesson.number, f"{chapter.idx}.{j}")
+
+	def test_get_tags(self):
+		tags = get_tags(self.course.name)
+		expected_tags = ["Frappe", "Learning", "Utility"]
+		self.assertEqual(set(tags), set(expected_tags))
+
+	def test_get_instructors(self):
+		instructors = get_instructors("LMS Course", self.course.name)
+		self.assertEqual(len(instructors), len(self.course.instructors))
+		self.assertEqual(instructors[0].name, "frappe@example.com")
+
+	def test_get_average_rating(self):
+		average_rating = get_average_rating(self.course.name)
+		self.assertEqual(average_rating, 4.5)
+
+	def add_rating(self, course_name, member, rating, review):
+		frappe.session.user = member
+		review = frappe.new_doc("LMS Course Review")
+		review.course = course_name
+		review.rating = rating
+		review.review = review
+		review.save()
+		frappe.session.user = "Administrator"
+
+	def test_get_reviews(self):
+		reviews = get_reviews(self.course.name)
+		self.assertEqual(len(reviews), 2)
+
+		for review in reviews:
+			if review.rating == 0.8:
+				self.assertEqual(review.member, self.student1.email)
+				self.assertEqual(review.review, "Good course")
+			elif review.rating == 1:
+				self.assertEqual(review.member, self.student2.email)
+				self.assertEqual(review.review, "Excellent course")
+
+	def tearDown(self):
+		if frappe.db.exists("LMS Course", self.course.name):
+			frappe.db.delete("LMS Enrollment", {"course": self.course.name})
+			frappe.db.delete("LMS Course Review", {"course": self.course.name})
+			frappe.db.delete("Course Lesson", {"course": self.course.name})
+			frappe.db.delete("Course Chapter", {"course": self.course.name})
+			frappe.delete_doc("LMS Course", self.course.name)
+
+		frappe.delete_doc("User", "student1@example.com")
+		frappe.delete_doc("User", "student2@example.com")
diff --git a/lms/lms/utils.py b/lms/lms/utils.py
index 44b4cb23..7ccd44af 100644
--- a/lms/lms/utils.py
+++ b/lms/lms/utils.py
@@ -1,8 +1,6 @@
 import hashlib
 import json
 import re
-import string
-from datetime import timedelta
 
 import frappe
 import requests
@@ -27,7 +25,7 @@ from frappe.utils import (
 	rounded,
 )
 
-from lms.lms.md import find_macros, markdown_to_html
+from lms.lms.md import find_macros
 
 RE_SLUG_NOTALLOWED = re.compile("[^a-z0-9]+")
 
@@ -83,6 +81,7 @@ def get_membership(course, member=None):
 				"current_lesson",
 				"progress",
 				"member",
+				"course",
 				"purchased_certificate",
 				"certificate",
 			],
@@ -150,6 +149,7 @@ def get_lesson_details(chapter, progress=False):
 				"file_type",
 				"instructor_notes",
 				"course",
+				"chapter",
 				"content",
 			],
 			as_dict=True,
@@ -228,16 +228,6 @@ def get_instructors(doctype, docname):
 	return instructor_details
 
 
-def get_students(course, batch=None):
-	"""Returns (email, full_name, username) of all the students of this batch as a list of dict."""
-	filters = {"course": course, "member_type": "Student"}
-
-	if batch:
-		filters["batch_old"] = batch
-
-	return frappe.get_all("LMS Enrollment", filters, ["member"])
-
-
 def get_average_rating(course):
 	ratings = [review.rating for review in get_reviews(course)]
 	if not len(ratings):
@@ -285,13 +275,6 @@ def get_sorted_reviews(course):
 	return rating_percent
 
 
-def is_certified(course):
-	certificate = frappe.get_all("LMS Certificate", {"member": frappe.session.user, "course": course})
-	if len(certificate):
-		return certificate[0].name
-	return
-
-
 def get_lesson_index(lesson_name):
 	"""Returns the {chapter_index}.{lesson_index} for the lesson."""
 	lesson = frappe.db.get_value("Lesson Reference", {"lesson": lesson_name}, ["idx", "parent"], as_dict=True)
@@ -311,14 +294,6 @@ def get_lesson_url(course, lesson_number):
 	return f"/lms/courses/{course}/learn/{lesson_number}"
 
 
-def get_batch(course, batch_name):
-	return frappe.get_all("LMS Batch Old", {"name": batch_name, "course": course})
-
-
-def get_slugified_chapter_title(chapter):
-	return slugify(chapter)
-
-
 def get_progress(course, lesson, member=None):
 	if not member:
 		member = frappe.session.user
@@ -330,52 +305,6 @@ def get_progress(course, lesson, member=None):
 	)
 
 
-def render_html(lesson):
-	youtube = lesson.youtube
-	quiz_id = lesson.quiz_id
-	body = lesson.body
-
-	if youtube and "/" in youtube:
-		youtube = youtube.split("/")[-1]
-
-	quiz_id = "{{ Quiz('" + quiz_id + "') }}" if quiz_id else ""
-	youtube = "{{ YouTubeVideo('" + youtube + "') }}" if youtube else ""
-	text = youtube + body + quiz_id
-
-	if lesson.question:
-		assignment = "{{ Assignment('" + lesson.question + "-" + lesson.file_type + "') }}"
-		text = text + assignment
-
-	return markdown_to_html(text)
-
-
-def is_mentor(course, email):
-	"""Checks if given user is a mentor for this course."""
-	if not email:
-		return False
-	return frappe.db.count("LMS Course Mentor Mapping", {"course": course, "mentor": email})
-
-
-def is_cohort_staff(course, user_email):
-	"""Returns True if the user is either a mentor or a staff for one or more active cohorts of this course."""
-	staff = {"doctype": "Cohort Staff", "course": course, "email": user_email}
-	mentor = {"doctype": "Cohort Mentor", "course": course, "email": user_email}
-	return frappe.db.exists(staff) or frappe.db.exists(mentor)
-
-
-def get_mentors(course):
-	"""Returns the list of all mentors for this course."""
-	course_mentors = []
-	mentors = frappe.get_all("LMS Course Mentor Mapping", {"course": course}, ["mentor"])
-	for mentor in mentors:
-		member = frappe.db.get_value("User", mentor.mentor, ["name", "username", "full_name", "user_image"])
-		member.batch_count = frappe.db.count(
-			"LMS Enrollment", {"member": member.name, "member_type": "Mentor"}
-		)
-		course_mentors.append(member)
-	return course_mentors
-
-
 def is_eligible_to_review(course):
 	"""Checks if user is eligible to review the course"""
 	if frappe.db.count("LMS Course Review", {"course": course, "owner": frappe.session.user}):
@@ -396,20 +325,6 @@ def get_course_progress(course, member=None):
 	return flt(((completed_lessons / lesson_count) * 100), precision)
 
 
-def get_initial_members(course):
-	members = frappe.get_all("LMS Enrollment", {"course": course}, ["member"], limit=3)
-
-	member_details = []
-	for member in members:
-		member_details.append(
-			frappe.db.get_value(
-				"User", member.member, ["name", "username", "full_name", "user_image"], as_dict=True
-			)
-		)
-
-	return member_details
-
-
 def is_instructor(course):
 	instructors = get_instructors("LMS Course", course)
 	for instructor in instructors:
@@ -418,57 +333,6 @@ def is_instructor(course):
 	return False
 
 
-def convert_number_to_character(number):
-	return string.ascii_uppercase[number]
-
-
-def get_signup_optin_checks():
-	mapper = frappe._dict(
-		{
-			"terms_of_use": {"page_name": "terms_page", "title": _("Terms of Use")},
-			"privacy_policy": {"page_name": "privacy_policy_page", "title": _("Privacy Policy")},
-			"cookie_policy": {"page_name": "cookie_policy_page", "title": _("Cookie Policy")},
-		}
-	)
-	checks = ["terms_of_use", "privacy_policy", "cookie_policy"]
-	links = []
-
-	for check in checks:
-		if frappe.db.get_single_value("LMS Settings", check):
-			page = frappe.db.get_single_value("LMS Settings", mapper[check].get("page_name"))
-			route = frappe.db.get_value("Web Page", page, "route")
-			links.append("<a href='/" + route + "'>" + mapper[check].get("title") + "</a>")
-
-	return (", ").join(links)
-
-
-def format_amount(amount, currency):
-	amount_reduced = amount / 1000
-	if amount_reduced < 1:
-		return fmt_money(amount, 0, currency)
-	precision = 0 if amount % 1000 == 0 else 1
-	return _("{0}k").format(fmt_money(amount_reduced, precision, currency))
-
-
-def format_number(number):
-	number_reduced = number / 1000
-	if number_reduced < 1:
-		return number
-	return f"{frappe.utils.flt(number_reduced, 1)}k"
-
-
-def first_lesson_exists(course):
-	first_chapter = frappe.db.get_value("Chapter Reference", {"parent": course, "idx": 1}, "name")
-	if not first_chapter:
-		return False
-
-	first_lesson = frappe.db.get_value("Lesson Reference", {"parent": first_chapter, "idx": 1}, "name")
-	if not first_lesson:
-		return False
-
-	return True
-
-
 def has_course_instructor_role(member=None):
 	return frappe.db.get_value(
 		"Has Role",
@@ -477,33 +341,6 @@ def has_course_instructor_role(member=None):
 	)
 
 
-def can_create_courses(course, member=None):
-	if not member:
-		member = frappe.session.user
-
-	instructors = frappe.get_all(
-		"Course Instructor",
-		{
-			"parent": course,
-		},
-		pluck="instructor",
-	)
-
-	if frappe.session.user == "Guest":
-		return False
-
-	if has_moderator_role(member):
-		return True
-
-	if has_course_instructor_role(member) and member in instructors:
-		return True
-
-	if not course and has_course_instructor_role(member):
-		return True
-
-	return False
-
-
 def can_create_batches(member=None):
 	if not member:
 		member = frappe.session.user
@@ -709,44 +546,6 @@ def get_lesson_count(course):
 	return lesson_count
 
 
-def get_all_memberships(member):
-	return frappe.get_all(
-		"LMS Enrollment",
-		{"member": member},
-		["name", "course", "batch_old", "current_lesson", "member_type", "progress"],
-	)
-
-
-def get_filtered_membership(course, memberships):
-	current_membership = list(filter(lambda x: x.course == course, memberships))
-	return current_membership[0] if len(current_membership) else None
-
-
-def show_start_learing_cta(course, membership):
-	if course.disable_self_learning or course.upcoming:
-		return False
-	if is_instructor(course.name):
-		return False
-	if course.status != "Approved":
-		return False
-	if not has_lessons(course):
-		return False
-	if not membership:
-		return True
-
-
-def has_lessons(course):
-	lesson_exists = False
-	chapter_exists = frappe.db.get_value(
-		"Chapter Reference", {"parent": course.name}, ["name", "chapter"], as_dict=True
-	)
-
-	if chapter_exists:
-		lesson_exists = frappe.db.exists("Lesson Reference", {"parent": chapter_exists.chapter})
-
-	return lesson_exists
-
-
 @frappe.whitelist(allow_guest=True)
 @rate_limit(limit=500, seconds=60 * 60)
 def get_chart_data(
@@ -770,7 +569,6 @@ def get_chart_data(
 	value_field = chart.value_based_on or "1"
 
 	filters = [([chart.document_type, "docstatus", "<", 2])]
-	print(chart.filters_json)
 	filters = filters + json.loads(chart.filters_json)
 	filters.append([doctype, datefield, ">=", from_date])
 	filters.append([doctype, datefield, "<=", to_date])
@@ -808,43 +606,6 @@ def get_course_completion_data():
 	]
 
 
-def get_telemetry_boot_info():
-	POSTHOG_PROJECT_FIELD = "posthog_project_id"
-	POSTHOG_HOST_FIELD = "posthog_host"
-
-	if not frappe.conf.get(POSTHOG_HOST_FIELD) or not frappe.conf.get(POSTHOG_PROJECT_FIELD):
-		return {}
-
-	return {
-		"posthog_host": frappe.conf.get(POSTHOG_HOST_FIELD),
-		"posthog_project_id": frappe.conf.get(POSTHOG_PROJECT_FIELD),
-		"enable_telemetry": 1,
-	}
-
-
-@frappe.whitelist()
-def is_onboarding_complete():
-	if not has_moderator_role():
-		return {"is_onboarded": True}
-
-	course_created = frappe.db.a_row_exists("LMS Course")
-	chapter_created = frappe.db.a_row_exists("Course Chapter")
-	lesson_created = frappe.db.a_row_exists("Course Lesson")
-
-	if course_created and chapter_created and lesson_created:
-		frappe.db.set_single_value("LMS Settings", "is_onboarding_complete", 1)
-
-	return {
-		"is_onboarded": frappe.db.get_single_value("LMS Settings", "is_onboarding_complete"),
-		"course_created": course_created,
-		"chapter_created": chapter_created,
-		"lesson_created": lesson_created,
-		"first_course": frappe.get_all("LMS Course", limit=1, order_by=None, pluck="name")[0]
-		if course_created
-		else None,
-	}
-
-
 def get_evaluator(course, batch=None):
 	evaluator = None
 	if batch:
@@ -965,13 +726,6 @@ def get_current_exchange_rate(source, target="USD"):
 	return details["rates"][target]
 
 
-@frappe.whitelist()
-def change_currency(amount, currency, country=None):
-	amount = cint(amount)
-	amount, currency = check_multicurrency(amount, currency, country)
-	return fmt_money(amount, 0, currency)
-
-
 @frappe.whitelist(allow_guest=True)
 @rate_limit(limit=500, seconds=60 * 60)
 def get_courses(filters=None, start=0):
@@ -1116,35 +870,11 @@ def get_course_fields():
 @frappe.whitelist(allow_guest=True)
 @rate_limit(limit=500, seconds=60 * 60)
 def get_course_details(course):
+	fields = get_course_fields()
 	course_details = frappe.db.get_value(
 		"LMS Course",
 		course,
-		[
-			"name",
-			"title",
-			"tags",
-			"description",
-			"image",
-			"video_link",
-			"short_introduction",
-			"published",
-			"upcoming",
-			"featured",
-			"disable_self_learning",
-			"published_on",
-			"category",
-			"status",
-			"paid_course",
-			"paid_certificate",
-			"course_price",
-			"currency",
-			"amount_usd",
-			"enable_certification",
-			"lessons",
-			"enrollments",
-			"rating",
-			"card_gradient",
-		],
+		fields,
 		as_dict=1,
 	)
 
@@ -2358,343 +2088,6 @@ def persona_captured():
 	frappe.db.set_single_value("LMS Settings", "persona_captured", 1)
 
 
-@frappe.whitelist()
-def get_my_courses():
-	my_courses = []
-	if frappe.session.user == "Guest":
-		return my_courses
-
-	courses = get_my_latest_courses()
-
-	if not len(courses):
-		courses = get_featured_home_courses()
-
-	if not len(courses):
-		courses = get_popular_courses()
-
-	for course in courses:
-		my_courses.append(get_course_details(course))
-
-	return my_courses
-
-
-def get_my_latest_courses():
-	return frappe.get_all(
-		"LMS Enrollment",
-		{
-			"member": frappe.session.user,
-		},
-		order_by="modified desc",
-		limit=3,
-		pluck="course",
-	)
-
-
-def get_featured_home_courses():
-	return frappe.get_all(
-		"LMS Course",
-		{"published": 1, "featured": 1},
-		order_by="published_on desc",
-		limit=3,
-		pluck="name",
-	)
-
-
-def get_popular_courses():
-	return frappe.get_all(
-		"LMS Course",
-		{
-			"published": 1,
-		},
-		order_by="enrollments desc",
-		limit=3,
-		pluck="name",
-	)
-
-
-@frappe.whitelist()
-def get_my_batches():
-	my_batches = []
-	if frappe.session.user == "Guest":
-		return my_batches
-
-	batches = get_my_latest_batches()
-
-	if not len(batches):
-		batches = get_upcoming_batches()
-
-	for batch in batches:
-		batch_details = get_batch_details(batch)
-		if batch_details:
-			my_batches.append(batch_details)
-
-	return my_batches
-
-
-def get_my_latest_batches():
-	return frappe.get_all(
-		"LMS Batch Enrollment",
-		{
-			"member": frappe.session.user,
-		},
-		order_by="creation desc",
-		limit=4,
-		pluck="batch",
-	)
-
-
-def get_upcoming_batches():
-	return frappe.get_all(
-		"LMS Batch",
-		{
-			"published": 1,
-			"start_date": [">=", getdate()],
-		},
-		order_by="start_date asc",
-		limit=4,
-		pluck="name",
-	)
-
-
-@frappe.whitelist()
-def get_my_live_classes():
-	my_live_classes = []
-	if frappe.session.user == "Guest":
-		return my_live_classes
-
-	batches = frappe.get_all(
-		"LMS Batch Enrollment",
-		{
-			"member": frappe.session.user,
-		},
-		order_by="creation desc",
-		pluck="batch",
-	)
-
-	live_class_details = frappe.get_all(
-		"LMS Live Class",
-		filters={
-			"date": [">=", getdate()],
-			"batch_name": ["in", batches],
-		},
-		fields=[
-			"name",
-			"title",
-			"description",
-			"time",
-			"date",
-			"duration",
-			"attendees",
-			"start_url",
-			"join_url",
-			"owner",
-		],
-		limit=2,
-		order_by="date",
-	)
-
-	if len(live_class_details):
-		for live_class in live_class_details:
-			live_class.course_title = frappe.db.get_value("LMS Course", live_class.course, "title")
-
-			my_live_classes.append(live_class)
-
-	return my_live_classes
-
-
-@frappe.whitelist()
-def get_created_courses():
-	created_courses = []
-	if frappe.session.user == "Guest":
-		return created_courses
-
-	CourseInstructor = frappe.qb.DocType("Course Instructor")
-	Course = frappe.qb.DocType("LMS Course")
-
-	query = (
-		frappe.qb.from_(CourseInstructor)
-		.join(Course)
-		.on(CourseInstructor.parent == Course.name)
-		.select(Course.name)
-		.where(CourseInstructor.instructor == frappe.session.user)
-		.orderby(Course.published_on, order=frappe.qb.desc)
-		.limit(3)
-	)
-
-	results = query.run(as_dict=True)
-	courses = [row["name"] for row in results]
-
-	for course in courses:
-		course_details = get_course_details(course)
-		created_courses.append(course_details)
-
-	return created_courses
-
-
-@frappe.whitelist()
-def get_created_batches():
-	created_batches = []
-	if frappe.session.user == "Guest":
-		return created_batches
-
-	CourseInstructor = frappe.qb.DocType("Course Instructor")
-	Batch = frappe.qb.DocType("LMS Batch")
-
-	query = (
-		frappe.qb.from_(CourseInstructor)
-		.join(Batch)
-		.on(CourseInstructor.parent == Batch.name)
-		.select(Batch.name)
-		.where(CourseInstructor.instructor == frappe.session.user)
-		.where(Batch.start_date >= getdate())
-		.orderby(Batch.start_date, order=frappe.qb.asc)
-		.limit(4)
-	)
-
-	results = query.run(as_dict=True)
-	batches = [row["name"] for row in results]
-
-	for batch in batches:
-		batch_details = get_batch_details(batch)
-		created_batches.append(batch_details)
-
-	return created_batches
-
-
-@frappe.whitelist()
-def get_admin_live_classes():
-	if frappe.session.user == "Guest":
-		return []
-
-	CourseInstructor = frappe.qb.DocType("Course Instructor")
-	LMSLiveClass = frappe.qb.DocType("LMS Live Class")
-
-	query = (
-		frappe.qb.from_(CourseInstructor)
-		.join(LMSLiveClass)
-		.on(CourseInstructor.parent == LMSLiveClass.batch_name)
-		.select(
-			LMSLiveClass.name,
-			LMSLiveClass.title,
-			LMSLiveClass.description,
-			LMSLiveClass.time,
-			LMSLiveClass.date,
-			LMSLiveClass.duration,
-			LMSLiveClass.attendees,
-			LMSLiveClass.start_url,
-			LMSLiveClass.join_url,
-			LMSLiveClass.owner,
-		)
-		.where(CourseInstructor.instructor == frappe.session.user)
-		.where(LMSLiveClass.date >= getdate())
-		.orderby(LMSLiveClass.date, order=frappe.qb.asc)
-		.limit(4)
-	)
-	results = query.run(as_dict=True)
-	return results
-
-
-@frappe.whitelist()
-def get_admin_evals():
-	if frappe.session.user == "Guest":
-		return []
-
-	evals = frappe.get_all(
-		"LMS Certificate Request",
-		{
-			"evaluator": frappe.session.user,
-			"date": [">=", getdate()],
-		},
-		[
-			"name",
-			"date",
-			"start_time",
-			"course",
-			"evaluator",
-			"google_meet_link",
-			"member",
-			"member_name",
-		],
-		limit=4,
-		order_by="date asc",
-	)
-
-	for evaluation in evals:
-		evaluation.course_title = frappe.db.get_value("LMS Course", evaluation.course, "title")
-
-	return evals
-
-
-def fetch_activity_dates(user):
-	doctypes = [
-		"LMS Course Progress",
-		"LMS Quiz Submission",
-		"LMS Assignment Submission",
-		"LMS Programming Exercise Submission",
-	]
-
-	all_dates = []
-	for dt in doctypes:
-		all_dates.extend(frappe.get_all(dt, {"member": user}, pluck="creation"))
-
-	return sorted({d.date() if hasattr(d, "date") else d for d in all_dates})
-
-
-def calculate_streaks(all_dates):
-	streak = 0
-	longest_streak = 0
-	prev_day = None
-
-	for d in all_dates:
-		if d.weekday() in (5, 6):
-			continue
-
-		if prev_day:
-			expected = prev_day + timedelta(days=1)
-			while expected.weekday() in (5, 6):
-				expected += timedelta(days=1)
-
-			streak = streak + 1 if d == expected else 1
-		else:
-			streak = 1
-
-		longest_streak = max(longest_streak, streak)
-		prev_day = d
-
-	return streak, longest_streak
-
-
-def calculate_current_streak(all_dates, streak):
-	if not all_dates:
-		return 0
-
-	last_date = all_dates[-1]
-	today = getdate()
-
-	ref_day = today
-	while ref_day.weekday() in (5, 6):
-		ref_day -= timedelta(days=1)
-
-	if last_date == ref_day or last_date == ref_day - timedelta(days=1):
-		return streak
-	return 0
-
-
-@frappe.whitelist()
-def get_streak_info():
-	if frappe.session.user == "Guest":
-		return {}
-
-	all_dates = fetch_activity_dates(frappe.session.user)
-	streak, longest_streak = calculate_streaks(all_dates)
-	current_streak = calculate_current_streak(all_dates, streak)
-
-	return {
-		"current_streak": current_streak,
-		"longest_streak": longest_streak,
-	}
-
-
 def validate_discussion_reply(doc, method):
 	topic = frappe.db.get_value(
 		"Discussion Topic", doc.topic, ["reference_doctype", "reference_docname"], as_dict=True
diff --git a/lms/templates/livecode/extension_footer.html b/lms/templates/livecode/extension_footer.html
deleted file mode 100644
index cc141598..00000000
--- a/lms/templates/livecode/extension_footer.html
+++ /dev/null
@@ -1,168 +0,0 @@
-<script type="text/javascript" src="/assets/frappe/node_modules/moment/min/moment-with-locales.min.js"></script>
-<script type="text/javascript" src="/assets/frappe/node_modules/moment-timezone/builds/moment-timezone-with-data.min.js"></script>
-<script type="text/javascript" src="/assets/frappe/js/frappe/utils/datetime.js"></script>
-
-<script type="text/javascript">
-    // comment_when is failing because of this
-    if (!frappe.sys_defaults) {
-      frappe.sys_defaults = {}
-    }
-</script>
-
-<script type="text/javascript" src="{{ livecode_url }}/static/livecode.js"></script>
-
-<script type="text/javascript" src="/assets/mon_school/js/livecode-files.js"></script>
-
-<template id="livecode-template">
-  <div class="livecode-editor livecode-editor-inline">
-    <div class="row">
-      <div class="col-lg-8 col-md-6">
-        <div class="controls">
-          <button class="run">{{ _("Run") }}</button>
-
-          <div class="exercise-controls pull-right">
-            <span style="padding-right: 10px;"><span class="last-submitted human-time" data-timestamp=""></span></span>
-            <button class="submit btn-primary">{{ _("Submit") }}</button>
-          </div>
-        </div>
-      </div>
-    </div>
-    <div class="code-editor">
-      <div class="row">
-        <div class="col-lg-8 col-md-6">
-          <div class="code-wrapper">
-            <textarea class="code"></textarea>
-          </div>
-        </div>
-        <div class="col-lg-4 col-md-6 canvas-wrapper">
-            <div class="svg-image" width="300" height="300"></div>
-            <pre class="output"></pre>
-        </div>
-      </div>
-    </div>
-  </div>
-</template>
-
-
-<script type="text/javascript">
-  function getLiveCodeOptions() {
-    return {
-      base_url: "{{ livecode_url }}",
-      runtime: "python",
-      files: LIVECODE_FILES, // loaded from livecode-files.js
-      command: ["python", "start.py"],
-      codemirror: true,
-      onMessage: {
-        image: function(editor, msg) {
-          const element = editor.parent.querySelector(".svg-image");
-          element.innerHTML = msg.image;
-        }
-      }
-    }
-  }
-
-  $(function() {
-    var editorLookup = {};
-
-    $("pre.example, pre.exercise").each((i, e) => {
-      var code = $(e).text();
-      var template = document.querySelector('#livecode-template');
-      var clone = template.content.cloneNode(true);
-
-      $(e)
-      .wrap('<div></div>')
-      .hide()
-      .parent()
-      .append(clone)
-      .find("textarea.code")
-      .val(code);
-
-      if ($(e).hasClass("exercise")) {
-        var last_submitted = $(e).data("last-submitted");
-        if (last_submitted) {
-          $(e).parent().find(".last-submitted")
-            .data("timestamp", last_submitted)
-            .html(__("Submitted {0}", [comment_when(last_submitted)]));
-        }
-      }
-      else {
-        $(e).parent().find(".exercise-controls").remove();
-      }
-
-      var editor = new LiveCodeEditor(e.parentElement, {
-        ...getLiveCodeOptions(),
-        codemirror: true,
-        onMessage: {
-          image: function(editor, msg) {
-            const canvasElement = editor.parent.querySelector("div.svg-image");
-            canvasElement.innerHTML = msg.image;
-          }
-        }
-      });
-
-      $(e).parent().find(".submit").on('click', function() {
-        var name = $(e).data("name");
-        let code = editor.codemirror.doc.getValue();
-
-        frappe.call("lms.lms.api.submit_solution", {
-          "exercise": name,
-          "code": code
-        }).then(r => {
-          if (r.message.name) {
-            frappe.msgprint("Submitted successfully!");
-
-            let d = r.message.creation;
-            $(e).parent().find(".human-time").html(__("Submitted {0}", [comment_when(d)]));
-          }
-        });
-      });
-    });
-
-    $(".exercise-image").each((i, e) => {
-      var svg = JSON.parse($(e).data("image"));
-      $(e).html(svg);
-    });
-
-    $("pre.exercise").each((i, e) => {
-      var svg = JSON.parse($(e).data("image"));
-      $(e).parent().find(".svg-image").html(svg);
-    });
-
-  });
-</script>
-
-<style type="text/css">
-  .svg-image {
-    border: 5px solid #ddd;
-    position: relative;
-    z-index: 0;
-    width: 310px;
-    height: 310px;
-  }
-  .livecode-editor {
-    margin-bottom: 30px;
-  }
-
-  .livecode-editor-small .svg-image {
-    border: 5px solid #ddd;
-    position: relative;
-    z-index: 0;
-    width: 210px;
-    height: 210px;
-  }
-
-  /* work-in-progress styles for showing admonition */
-  .admonition {
-    border: 1px solid #aaa;
-    border-left: .5rem solid #888;
-    border-radius: .3em;
-    font-size: 0.9em;
-    margin: 1.5em 0;
-    padding: 0 0.5em;
-  }
-  .admonition-title {
-    padding: 0.5em 0px;
-    font-weight: bold;
-    padding-top:
-  }
-</style>
diff --git a/lms/templates/livecode/extension_header.html b/lms/templates/livecode/extension_header.html
deleted file mode 100644
index 1499d750..00000000
--- a/lms/templates/livecode/extension_header.html
+++ /dev/null
@@ -1,8 +0,0 @@
-<link rel="stylesheet" href="{{ livecode_url }}/static/codemirror/lib/codemirror.css">
-
-<script src="{{ livecode_url }}/static/codemirror/lib/codemirror.js"></script>
-<script src="{{ livecode_url }}/static/codemirror/mode/python/python.js"></script>
-<script src="{{ livecode_url }}/static/codemirror/keymap/sublime.js"></script>
-
-<script src="{{ livecode_url }}/static/codemirror/addon/edit/matchbrackets.js"></script>
-<script src="{{ livecode_url }}/static/codemirror/addon/comment/comment.js"></script>

From 196d4a835fc714b23ae69f258f3584aa952d44c8 Mon Sep 17 00:00:00 2001
From: Jannat Patel <pateljannat2308@gmail.com>
Date: Mon, 8 Dec 2025 21:10:13 +0530
Subject: [PATCH 04/14] test: utils

---
 .../lms_certificate/test_lms_certificate.py   |  31 +---
 lms/lms/doctype/lms_course/test_lms_course.py |   4 +-
 .../lms_enrollment/lms_enrollment.json        |   3 +-
 .../doctype/lms_enrollment/lms_enrollment.py  |  77 +++-----
 .../lms_enrollment/test_lms_enrollment.py     |  15 +-
 lms/lms/test_utils.py                         |  37 ++--
 lms/lms/utils.py                              |  18 +-
 lms/templates/reviews.html                    | 167 ------------------
 8 files changed, 56 insertions(+), 296 deletions(-)
 delete mode 100644 lms/templates/reviews.html

diff --git a/lms/lms/doctype/lms_certificate/test_lms_certificate.py b/lms/lms/doctype/lms_certificate/test_lms_certificate.py
index 81cf129b..04842eb9 100644
--- a/lms/lms/doctype/lms_certificate/test_lms_certificate.py
+++ b/lms/lms/doctype/lms_certificate/test_lms_certificate.py
@@ -3,35 +3,6 @@
 
 import unittest
 
-import frappe
-from frappe.utils import cint, nowdate
-
-from lms.lms.doctype.lms_certificate.lms_certificate import create_certificate
-from lms.lms.doctype.lms_course.test_lms_course import new_course
-
 
 class TestLMSCertificate(unittest.TestCase):
-	def test_certificate_creation(self):
-		course = new_course(
-			"Test Certificate",
-			{
-				"enable_certification": 1,
-			},
-		)
-		create_enrollment(course.name)
-		certificate = create_certificate(course.name)
-
-		self.assertEqual(certificate.member, "Administrator")
-		self.assertEqual(certificate.course, course.name)
-		self.assertEqual(certificate.issue_date, nowdate())
-
-		frappe.db.delete("LMS Certificate", certificate.name)
-		frappe.db.delete("LMS Course", course.name)
-
-
-def create_enrollment(course):
-	enrollment = frappe.new_doc("LMS Enrollment")
-	enrollment.course = course
-	enrollment.member = frappe.session.user
-	enrollment.progress = cint(100)
-	enrollment.save()
+	pass
diff --git a/lms/lms/doctype/lms_course/test_lms_course.py b/lms/lms/doctype/lms_course/test_lms_course.py
index 72d7dadc..936fb0e9 100644
--- a/lms/lms/doctype/lms_course/test_lms_course.py
+++ b/lms/lms/doctype/lms_course/test_lms_course.py
@@ -19,9 +19,6 @@ class TestLMSCourse(unittest.TestCase):
 
 		if frappe.db.exists("LMS Course", "test-course"):
 			frappe.db.delete("Batch Course", {"course": "test-course"})
-			frappe.db.delete("Exercise Submission", {"course": "test-course"})
-			frappe.db.delete("Exercise Latest Submission", {"course": "test-course"})
-			frappe.db.delete("LMS Exercise", {"course": "test-course"})
 			frappe.db.delete("LMS Enrollment", {"course": "test-course"})
 			frappe.db.delete("Course Lesson", {"course": "test-course"})
 			frappe.db.delete("Course Chapter", {"course": "test-course"})
@@ -67,6 +64,7 @@ def new_course(title, additional_filters=None):
 			"video_link": "https://youtu.be/pEbIhUySqbk",
 			"image": "/assets/lms/images/course-home.png",
 			"instructors": [{"instructor": user}],
+			"published": 1,
 		}
 
 		if additional_filters:
diff --git a/lms/lms/doctype/lms_enrollment/lms_enrollment.json b/lms/lms/doctype/lms_enrollment/lms_enrollment.json
index 887724f2..a3863f33 100644
--- a/lms/lms/doctype/lms_enrollment/lms_enrollment.json
+++ b/lms/lms/doctype/lms_enrollment/lms_enrollment.json
@@ -21,7 +21,6 @@
   "section_break_8",
   "cohort",
   "subgroup",
-  "batch_old",
   "column_break_12",
   "member_type",
   "role"
@@ -149,7 +148,7 @@
  "grid_page_length": 50,
  "index_web_pages_for_search": 1,
  "links": [],
- "modified": "2025-12-08 21:27:30.733482",
+ "modified": "2025-12-09 21:27:30.733482",
  "modified_by": "Administrator",
  "module": "LMS",
  "name": "LMS Enrollment",
diff --git a/lms/lms/doctype/lms_enrollment/lms_enrollment.py b/lms/lms/doctype/lms_enrollment/lms_enrollment.py
index fbf6994f..e896edcc 100644
--- a/lms/lms/doctype/lms_enrollment/lms_enrollment.py
+++ b/lms/lms/doctype/lms_enrollment/lms_enrollment.py
@@ -9,28 +9,43 @@ from frappe.utils import ceil
 
 class LMSEnrollment(Document):
 	def validate(self):
-		self.validate_membership_in_same_batch()
+		self.validate_course_enrollment_eligibility()
 
 	def on_update(self):
 		update_program_progress(self.member)
 
-	def validate_membership_in_same_batch(self):
-		filters = {"member": self.member, "course": self.course, "name": ["!=", self.name]}
-		if self.batch_old:
-			filters["batch_old"] = self.batch_old
-		previous_membership = frappe.db.get_value(
-			"LMS Enrollment", filters, fieldname=["member_type", "member"], as_dict=1
+	def validate_course_enrollment_eligibility(self):
+		course_details = frappe.db.get_value(
+			"LMS Course",
+			self.course,
+			["published", "disable_self_learning", "paid_course", "paid_certificate"],
+			as_dict=True,
 		)
 
-		if previous_membership:
-			member_name = frappe.db.get_value("User", self.member, "full_name")
-			course_title = frappe.db.get_value("LMS Course", self.course, "title")
+		if course_details.disable_self_learning:
 			frappe.throw(
-				_("{0} is already a {1} of the course {2}").format(
-					member_name, previous_membership.member_type, course_title
+				_(
+					"You cannot enroll in this course as self-learning is disabled. Please contact the Administrator."
 				)
 			)
 
+		if not course_details.published:
+			frappe.throw(_("You cannot enroll in an unpublished course."))
+
+		if course_details.paid_course:
+			payment = frappe.db.exists(
+				"LMS Payment",
+				{
+					"reference_doctype": "LMS Course",
+					"reference_docname": course,
+					"member": member,
+					"payment_receipt": True,
+				},
+			)
+
+			if not payment:
+				frappe.throw(_("You need to complete the payment for this course before enrolling."))
+
 
 def update_program_progress(member):
 	programs = frappe.get_all("LMS Program Member", {"member": member}, ["parent", "name"])
@@ -49,8 +64,6 @@ def update_program_progress(member):
 
 @frappe.whitelist()
 def create_membership(course, batch=None, member=None, member_type="Student", role="Member"):
-	validate_course_enrollment_eligibility(course, member)
-
 	enrollment = frappe.new_doc("LMS Enrollment")
 	enrollment.update(
 		{
@@ -66,42 +79,6 @@ def create_membership(course, batch=None, member=None, member_type="Student", ro
 	return enrollment
 
 
-def validate_course_enrollment_eligibility(course, member):
-	if not member:
-		member = frappe.session.user
-
-	course_details = frappe.db.get_value(
-		"LMS Course",
-		course,
-		["published", "disable_self_learning", "paid_course", "paid_certificate"],
-		as_dict=True,
-	)
-
-	if course_details.disable_self_learning:
-		frappe.throw(
-			_(
-				"You cannot enroll in this course as self-learning is disabled. Please contact the Administrator."
-			)
-		)
-
-	if not course_details.published:
-		frappe.throw(_("You cannot enroll in an unpublished course."))
-
-	if course_details.paid_course:
-		payment = frappe.db.exists(
-			"LMS Payment",
-			{
-				"reference_doctype": "LMS Course",
-				"reference_docname": course,
-				"member": member,
-				"payment_receipt": True,
-			},
-		)
-
-		if not payment:
-			frappe.throw(_("You need to complete the payment for this course before enrolling."))
-
-
 @frappe.whitelist()
 def update_current_membership(batch, course, member):
 	all_memberships = frappe.get_all("LMS Enrollment", {"member": member, "course": course})
diff --git a/lms/lms/doctype/lms_enrollment/test_lms_enrollment.py b/lms/lms/doctype/lms_enrollment/test_lms_enrollment.py
index 22e7fb4c..acee732c 100644
--- a/lms/lms/doctype/lms_enrollment/test_lms_enrollment.py
+++ b/lms/lms/doctype/lms_enrollment/test_lms_enrollment.py
@@ -5,19 +5,6 @@ import unittest
 
 import frappe
 
-from lms.lms.doctype.lms_course.test_lms_course import new_course, new_user
-
 
 class TestLMSEnrollment(unittest.TestCase):
-	def test_membership(self):
-		course = new_course("Test Enrollment")
-		enrollment = frappe.new_doc("LMS Enrollment")
-		enrollment.course = course.name
-		enrollment.member = frappe.session.user
-
-		enrollment.save()
-
-		self.assertEqual(enrollment.course, course.name)
-		self.assertEqual(enrollment.member, "Administrator")
-		frappe.db.delete("LMS Enrollment", enrollment.name)
-		frappe.db.delete("LMS Course", course.name)
+	pass
diff --git a/lms/lms/test_utils.py b/lms/lms/test_utils.py
index ecec9f3e..344667f4 100644
--- a/lms/lms/test_utils.py
+++ b/lms/lms/test_utils.py
@@ -6,6 +6,7 @@ from .utils import (
 	get_average_rating,
 	get_chapters,
 	get_instructors,
+	get_lesson_index,
 	get_lessons,
 	get_membership,
 	get_reviews,
@@ -16,13 +17,14 @@ from .utils import (
 
 class TestUtils(unittest.TestCase):
 	def setUp(self):
+		self.student1 = self.create_user("student1@example.com", "Ashley", "Smith", "LMS Student")
+		self.student2 = self.create_user("student2@example.com", "John", "Doe", "LMS Student")
+		self.admin = self.create_user("frappe@example.com", "Frappe", "Admin", "Moderator")
+
 		self.create_a_course()
 		self.add_chapters()
 		self.add_lessons()
 
-		self.student1 = self.create_student("student1@example.com", "Ashley", "Smith")
-		self.student2 = self.create_student("student2@example.com", "John", "Doe")
-
 		self.add_enrollment(self.course.name, self.student1.email)
 		self.add_enrollment(self.course.name, self.student2.email)
 
@@ -73,15 +75,18 @@ class TestUtils(unittest.TestCase):
 				chapterDoc.append("lessons", {"lesson": lesson.name})
 			chapterDoc.save()
 
-	def create_student(self, email, first_name, last_name):
-		student = frappe.new_doc("User")
-		student.email = email
-		student.first_name = first_name
-		student.last_name = last_name
-		student.user_type = "Website User"
-		student.append("roles", {"role": "LMS Student"})
-		student.save()
-		return student
+	def create_user(self, email, first_name, last_name, role):
+		if not frappe.db.exists("User", email):
+			student = frappe.new_doc("User")
+			student.email = email
+			student.first_name = first_name
+			student.last_name = last_name
+			student.user_type = "Website User"
+			student.append("roles", {"role": role})
+			student.save()
+			return student
+		else:
+			return frappe.get_doc("User", email)
 
 	def test_simple_slugs(self):
 		self.assertEqual(slugify("hello-world"), "hello-world")
@@ -121,7 +126,7 @@ class TestUtils(unittest.TestCase):
 			self.assertEqual(len(chapter_lessons), 2)
 			for j, lesson in enumerate(chapter_lessons, start=1):
 				self.assertEqual(lesson.title, f"Lesson {j} of {chapter.chapter}")
-				self.assertEqual(lesson.number, f"{chapter.idx}.{j}")
+				self.assertEqual(lesson.number, f"{chapter.idx}-{j}")
 
 	def test_get_tags(self):
 		tags = get_tags(self.course.name)
@@ -158,6 +163,11 @@ class TestUtils(unittest.TestCase):
 				self.assertEqual(review.member, self.student2.email)
 				self.assertEqual(review.review, "Excellent course")
 
+	def test_get_lesson_index(self):
+		lessons = get_lessons(self.course.name)
+		for lesson in lessons:
+			self.assertEqual(get_lesson_index(lesson.name), lesson.number)
+
 	def tearDown(self):
 		if frappe.db.exists("LMS Course", self.course.name):
 			frappe.db.delete("LMS Enrollment", {"course": self.course.name})
@@ -168,3 +178,4 @@ class TestUtils(unittest.TestCase):
 
 		frappe.delete_doc("User", "student1@example.com")
 		frappe.delete_doc("User", "student2@example.com")
+		frappe.delete_doc("User", "frappe@example.com")
diff --git a/lms/lms/utils.py b/lms/lms/utils.py
index 7ccd44af..ba4d75f5 100644
--- a/lms/lms/utils.py
+++ b/lms/lms/utils.py
@@ -154,7 +154,7 @@ def get_lesson_details(chapter, progress=False):
 			],
 			as_dict=True,
 		)
-		lesson_details.number = f"{chapter.idx}.{row.idx}"
+		lesson_details.number = f"{chapter.idx}-{row.idx}"
 		lesson_details.icon = get_lesson_icon(lesson_details.body, lesson_details.content)
 
 		if progress:
@@ -259,22 +259,6 @@ def get_reviews(course):
 	return reviews
 
 
-def get_sorted_reviews(course):
-	rating_count = rating_percent = frappe._dict()
-	keys = ["5.0", "4.0", "3.0", "2.0", "1.0"]
-	for key in keys:
-		rating_count[key] = 0
-
-	reviews = get_reviews(course)
-	for review in reviews:
-		rating_count[cstr(review.rating)] += 1
-
-	for key in keys:
-		rating_percent[key] = rating_count[key] / len(reviews) * 100
-
-	return rating_percent
-
-
 def get_lesson_index(lesson_name):
 	"""Returns the {chapter_index}.{lesson_index} for the lesson."""
 	lesson = frappe.db.get_value("Lesson Reference", {"lesson": lesson_name}, ["idx", "parent"], as_dict=True)
diff --git a/lms/templates/reviews.html b/lms/templates/reviews.html
deleted file mode 100644
index e6c1a6be..00000000
--- a/lms/templates/reviews.html
+++ /dev/null
@@ -1,167 +0,0 @@
-{% if not course.upcoming %}
-<div class="reviews-parent">
-    {% set reviews = get_reviews(course.name) %}
-    <div class="page-title mb-5"> {{ _("Reviews") }} </div>
-
-
-    {% if avg_rating %}
-    <div class="reviews-header">
-        <div class="text-center">
-            <div class="avg-rating">
-                {{ frappe.utils.flt(avg_rating, frappe.get_system_settings("float_precision") or 3) }}
-            </div>
-
-
-            <div class="avg-rating-stars">
-                <div class="rating">
-                    {% for i in [1, 2, 3, 4, 5] %}
-                    <svg class="icon icon-lg {% if i <= frappe.utils.ceil(avg_rating) %} star-click {% endif %}" data-rating="{{ i }}">
-                        <use href="#icon-star"></use>
-                    </svg>
-                    {% endfor %}
-                </div>
-            </div>
-
-            <div class="course-meta"> {{ reviews | length }} {{ _("ratings") }} </div>
-            <!--
-
-             -->
-
-            <div class="mt-5">
-                {% include "lms/templates/reviews_cta.html" %}
-            </div>
-        </div>
-
-
-        <div class="vertical-divider"></div>
-        {% set sorted_reviews = get_sorted_reviews(course.name) %}
-        <div>
-            {% for review in sorted_reviews %}
-            <div class="d-flex align-items-center mb-3">
-                <div class="course-meta mr-2">
-                    {{ frappe.utils.cint(review) }} {{ _("stars") }}
-                </div>
-                <div class="progress">
-                    <div class="progress-bar" role="progressbar" aria-valuenow="{{ sorted_reviews[review] }}"
-                        aria-valuemin="0" aria-valuemax="100" style="width:{{ sorted_reviews[review] }}%">
-                        <span class="sr-only"> {{ sorted_reviews[review] }} {{ _("Complete") }} </span>
-                    </div>
-                </div>
-                <div class="course-meta ml-3"> {{ frappe.utils.cint(sorted_reviews[review]) }}% </div>
-            </div>
-            {% endfor %}
-        </div>
-    </div>
-    {% endif %}
-
-
-    {% if reviews | length %}
-    <div class="mt-12">
-        {% for review in reviews %}
-        <div class="mb-4">
-            <div class="d-flex align-items-center">
-                <div class="mr-4">
-                {{ widgets.Avatar(member=review.owner_details, avatar_class="avatar-medium") }}
-                </div>
-                <div>
-                    <div class="d-flex align-items-center">
-                        <a class="button-links mr-4" href="/lms/users/{{ review.owner_details.username }}">
-                        <span class="bold-heading">
-                            {{ review.owner_details.full_name }}
-                        </span>
-                        </a>
-                        <div class="frappe-timestamp course-meta" data-timestamp="{{ review.creation }}">
-                            {{ review.creation }}
-                        </div>
-                    </div>
-                    <div class="rating">
-                        {% for i in [1, 2, 3, 4, 5] %}
-                            <svg class="icon icon-md {% if i <= review.rating %} star-click {% endif %}" data-rating="{{ i }}">
-                                <use href="#icon-star"></use>
-                            </svg>
-                        {% endfor %}
-                    </div>
-                </div>
-            </div>
-            <div class="review-content"> {{ review.review }} </div>
-        </div>
-        {% if loop.index != reviews | length %}
-            <div class="card-divider"></div>
-        {% endif %}
-        {% endfor %}
-    </div>
-
-
-    {% else %}
-    <div class="empty-state">
-        <img class="icon icon-xl" src="/assets/lms/icons/comment.svg">
-        <div class="empty-state-text">
-            <div class="empty-state-heading">{{ _("Review the course") }}</div>
-            <div class="course-meta">{{ _("Help us improve our course material.") }}</div>
-            <div class="mt-2">
-                {% include "lms/templates/reviews_cta.html" %}
-            </div>
-        </div>
-    </div>
-    {% endif %}
-</div>
-
-
-<div class="modal fade review-modal" id="review-modal" tabindex="-1" role="dialog"
-    aria-labelledby="exampleModalLabel" aria-hidden="true">
-    <div class="modal-dialog" role="document">
-        <div class="modal-content">
-            <div class="modal-header">
-                <div class="modal-title">{{ _("Write a Review") }}</div>
-            </div>
-
-
-            <div class="modal-body">
-                <form class="review-form" id="review-form">
-                    <div class="form-group">
-                        <div class="clearfix">
-                            <label class="control-label reqd" style="padding-right: 0px;">{{ _("Rating") }}</label>
-                        </div>
-                        <div class="control-input-wrapper">
-                            <div class="control-input">
-                                <div class="rating rating-field" id="rating">
-                                    {% for i in [1, 2, 3, 4, 5] %}
-                                    <svg class="icon icon-md icon-rating" data-rating="{{ i }}">
-                                        <use href="#icon-star"></use>
-                                    </svg>
-                                    {% endfor %}
-                                </div>
-                            </div>
-                        </div>
-                    </div>
-
-
-                    <div class="form-group">
-                        <div class="clearfix">
-                            <label class="control-label reqd" style="padding-right: 0px;">{{ _("Review") }}</label>
-                        </div>
-                        <div class="control-input-wrapper">
-                            <div class="control-input">
-                                <textarea type="text" autocomplete="off" class="input-with-feedback form-control review-field"
-                                data-fieldtype="Text" data-fieldname="feedback_comments" spellcheck="false"></textarea>
-                            </div>
-                        </div>
-                    </div>
-                    <p class="error-field muted-text"></p>
-                </form>
-            </div>
-
-
-            <div class="modal-footer">
-                <button class="btn btn-secondary btn-sm mr-2" data-dismiss="modal" aria-label="Close">
-                    {{ _("Discard") }}
-                </button>
-
-                <button class="btn btn-primary btn-sm" data-course="{{ course.name | urlencode}}" id="submit-review">
-                    {{ _("Submit") }}
-                </button>
-            </div>
-        </div>
-    </div>
-</div>
-{% endif %}

From c901a159699c6f13a262bc6beada71c1c65160a4 Mon Sep 17 00:00:00 2001
From: Jannat Patel <pateljannat2308@gmail.com>
Date: Tue, 9 Dec 2025 11:24:35 +0530
Subject: [PATCH 05/14] test: utils

---
 lms/lms/test_utils.py          | 61 +++++++++++++++++++++++++++-------
 lms/lms/utils.py               |  8 -----
 lms/templates/reviews_cta.html |  9 -----
 3 files changed, 49 insertions(+), 29 deletions(-)
 delete mode 100644 lms/templates/reviews_cta.html

diff --git a/lms/lms/test_utils.py b/lms/lms/test_utils.py
index 344667f4..35e00d47 100644
--- a/lms/lms/test_utils.py
+++ b/lms/lms/test_utils.py
@@ -7,19 +7,27 @@ from .utils import (
 	get_chapters,
 	get_instructors,
 	get_lesson_index,
+	get_lesson_url,
 	get_lessons,
 	get_membership,
 	get_reviews,
 	get_tags,
+	has_course_instructor_role,
+	has_evaluator_role,
+	has_moderator_role,
+	has_student_role,
+	is_instructor,
 	slugify,
 )
 
 
 class TestUtils(unittest.TestCase):
 	def setUp(self):
-		self.student1 = self.create_user("student1@example.com", "Ashley", "Smith", "LMS Student")
-		self.student2 = self.create_user("student2@example.com", "John", "Doe", "LMS Student")
-		self.admin = self.create_user("frappe@example.com", "Frappe", "Admin", "Moderator")
+		self.student1 = self.create_user("student1@example.com", "Ashley", "Smith", ["LMS Student"])
+		self.student2 = self.create_user("student2@example.com", "John", "Doe", ["LMS Student"])
+		self.admin = self.create_user(
+			"frappe@example.com", "Frappe", "Admin", ["Moderator", "Course Creator", "Batch Evaluator"]
+		)
 
 		self.create_a_course()
 		self.add_chapters()
@@ -75,16 +83,17 @@ class TestUtils(unittest.TestCase):
 				chapterDoc.append("lessons", {"lesson": lesson.name})
 			chapterDoc.save()
 
-	def create_user(self, email, first_name, last_name, role):
+	def create_user(self, email, first_name, last_name, roles):
 		if not frappe.db.exists("User", email):
-			student = frappe.new_doc("User")
-			student.email = email
-			student.first_name = first_name
-			student.last_name = last_name
-			student.user_type = "Website User"
-			student.append("roles", {"role": role})
-			student.save()
-			return student
+			user = frappe.new_doc("User")
+			user.email = email
+			user.first_name = first_name
+			user.last_name = last_name
+			user.user_type = "Website User"
+			for role in roles:
+				user.append("roles", {"role": role})
+			user.save()
+			return user
 		else:
 			return frappe.get_doc("User", email)
 
@@ -168,6 +177,34 @@ class TestUtils(unittest.TestCase):
 		for lesson in lessons:
 			self.assertEqual(get_lesson_index(lesson.name), lesson.number)
 
+	def test_get_lesson_url(self):
+		lessons = get_lessons(self.course.name)
+		for lesson in lessons:
+			expected_url = f"/lms/courses/{self.course.name}/learn/{lesson.number}"
+			self.assertEqual(get_lesson_url(self.course.name, lesson.number), expected_url)
+
+	def test_is_instructor(self):
+		frappe.session.user = "frappe@example.com"
+		self.assertTrue(is_instructor(self.course.name))
+		frappe.session.user = "Administrator"
+		self.assertFalse(is_instructor(self.course.name))
+
+	def test_has_course_instructor_role(self):
+		self.assertIsNotNone(has_course_instructor_role("frappe@example.com"))
+		self.assertIsNone(has_course_instructor_role("student1@example.com"))
+
+	def test_has_moderator_role(self):
+		self.assertIsNotNone(has_moderator_role("frappe@example.com"))
+		self.assertIsNone(has_moderator_role("student2@example.com"))
+
+	def test_has_evaluator_role(self):
+		self.assertIsNotNone(has_evaluator_role("frappe@example.com"))
+		self.assertIsNone(has_evaluator_role("student2@example.com"))
+
+	def test_has_student_role(self):
+		self.assertIsNotNone(has_student_role("student1@example.com"))
+		self.assertIsNotNone(has_student_role("student2@example.com"))
+
 	def tearDown(self):
 		if frappe.db.exists("LMS Course", self.course.name):
 			frappe.db.delete("LMS Enrollment", {"course": self.course.name})
diff --git a/lms/lms/utils.py b/lms/lms/utils.py
index ba4d75f5..5d4bfae8 100644
--- a/lms/lms/utils.py
+++ b/lms/lms/utils.py
@@ -12,7 +12,6 @@ from frappe.rate_limiter import rate_limit
 from frappe.utils import (
 	add_months,
 	cint,
-	cstr,
 	flt,
 	fmt_money,
 	format_datetime,
@@ -289,13 +288,6 @@ def get_progress(course, lesson, member=None):
 	)
 
 
-def is_eligible_to_review(course):
-	"""Checks if user is eligible to review the course"""
-	if frappe.db.count("LMS Course Review", {"course": course, "owner": frappe.session.user}):
-		return False
-	return True
-
-
 def get_course_progress(course, member=None):
 	"""Returns the course progress of the session user"""
 	lesson_count = get_lessons(course, get_details=False)
diff --git a/lms/templates/reviews_cta.html b/lms/templates/reviews_cta.html
deleted file mode 100644
index f10250ae..00000000
--- a/lms/templates/reviews_cta.html
+++ /dev/null
@@ -1,9 +0,0 @@
-{% if membership and is_eligible_to_review(course.name) %}
-    <span class="btn btn-secondary btn-sm review-link">
-        {{ _("Write a review") }}
-    </span>
-{% elif not is_instructor and frappe.session.user == "Guest" %}
-    <a class="btn btn-secondary btn-sm" href="/login?redirect-to=/courses/{{ course.name }}">
-        {{ _("Write a review") }}
-    </a>
-{% endif %}

From 7e3c5beaea56ee62b60cd49bc0317c8a182ea7ab Mon Sep 17 00:00:00 2001
From: Jannat Patel <pateljannat2308@gmail.com>
Date: Wed, 10 Dec 2025 11:17:21 +0530
Subject: [PATCH 06/14] fix: filter search records based on roles

---
 frontend/components.d.ts                      |  2 +
 .../CommandPalette/CommandPalette.vue         | 21 +++++++--
 .../CommandPalette/CommandPaletteGroup.vue    |  5 +-
 .../src/components/Sidebar/AppSidebar.vue     |  7 ++-
 frontend/src/components/Sidebar/Apps.vue      |  2 +-
 frontend/vite.config.js                       |  2 +-
 lms/command_palette.py                        | 43 +++++++++++++++--
 lms/sqlite.py                                 | 46 ++++++-------------
 8 files changed, 81 insertions(+), 47 deletions(-)

diff --git a/frontend/components.d.ts b/frontend/components.d.ts
index f7cc0a5d..95951283 100644
--- a/frontend/components.d.ts
+++ b/frontend/components.d.ts
@@ -90,6 +90,8 @@ declare module 'vue' {
     LucideArrowDown: typeof import('~icons/lucide/arrow-down')['default']
     LucideArrowUp: typeof import('~icons/lucide/arrow-up')['default']
     LucideCornerDownLeft: typeof import('~icons/lucide/corner-down-left')['default']
+    LucideSearch: typeof import('~icons/lucide/search')['default']
+    LucideX: typeof import('~icons/lucide/x')['default']
     Members: typeof import('./src/components/Settings/Members.vue')['default']
     MobileLayout: typeof import('./src/components/MobileLayout.vue')['default']
     MultiSelect: typeof import('./src/components/Controls/MultiSelect.vue')['default']
diff --git a/frontend/src/components/CommandPalette/CommandPalette.vue b/frontend/src/components/CommandPalette/CommandPalette.vue
index c4fdf036..27e6eb4b 100644
--- a/frontend/src/components/CommandPalette/CommandPalette.vue
+++ b/frontend/src/components/CommandPalette/CommandPalette.vue
@@ -18,11 +18,17 @@
 
 				<div class="max-h-96 overflow-auto mb-2">
 					<div v-if="query.length" class="mt-5 space-y-5">
-						<CommandPaletteGroup :list="searchResults" />
+						<CommandPaletteGroup
+							:list="searchResults"
+							@navigateTo="navigateTo"
+						/>
 					</div>
 
 					<div v-else class="mt-5 space-y-5">
-						<CommandPaletteGroup :list="jumpToOptions" />
+						<CommandPaletteGroup
+							:list="jumpToOptions"
+							@navigateTo="navigateTo"
+						/>
 					</div>
 				</div>
 
@@ -61,7 +67,8 @@
 </template>
 <script setup lang="ts">
 import { createResource, debounce, Dialog } from 'frappe-ui'
-import { computed, inject, nextTick, onMounted, ref, watch } from 'vue'
+import { nextTick, onMounted, ref, watch } from 'vue'
+import { useRouter } from 'vue-router'
 import {
 	BookOpen,
 	Briefcase,
@@ -72,7 +79,6 @@ import {
 	Search,
 	Users,
 } from 'lucide-vue-next'
-import { useRouter } from 'vue-router'
 import CommandPaletteGroup from './CommandPaletteGroup.vue'
 
 const show = defineModel<boolean>({ required: true, default: false })
@@ -146,6 +152,13 @@ watch(
 	{ immediate: true }
 )
 
+watch(show, () => {
+	if (!show.value) {
+		query.value = ''
+		searchResults.value = []
+	}
+})
+
 onMounted(() => {
 	addKeyboardShortcuts()
 })
diff --git a/frontend/src/components/CommandPalette/CommandPaletteGroup.vue b/frontend/src/components/CommandPalette/CommandPaletteGroup.vue
index cf34b593..715b53ea 100644
--- a/frontend/src/components/CommandPalette/CommandPaletteGroup.vue
+++ b/frontend/src/components/CommandPalette/CommandPaletteGroup.vue
@@ -8,6 +8,7 @@
 				v-for="item in result.items"
 				class="flex items-center justify-between p-2 rounded hover:bg-surface-gray-2 cursor-pointer"
 				:class="{ 'bg-surface-gray-2': item.isActive }"
+				@click="emit('navigateTo', item.route)"
 			>
 				<div class="flex items-center space-x-3">
 					<component
@@ -18,7 +19,7 @@
 					<div v-html="item.title"></div>
 				</div>
 				<div v-if="item.modified" class="text-ink-gray-5">
-					{{ item.modified }} {{ dayjs.unix(item.modified).fromNow(true) }}
+					{{ dayjs.unix(item.modified).fromNow(true) }}
 				</div>
 			</div>
 		</div>
@@ -26,9 +27,9 @@
 </template>
 <script lang="ts" setup>
 import { inject } from 'vue'
-import { dayjsLocal } from 'frappe-ui'
 
 const dayjs = inject<any>('$dayjs')
+const emit = defineEmits(['navigateTo'])
 
 const props = defineProps<{
 	list: Array<{
diff --git a/frontend/src/components/Sidebar/AppSidebar.vue b/frontend/src/components/Sidebar/AppSidebar.vue
index 8a9a482d..c3e764fc 100644
--- a/frontend/src/components/Sidebar/AppSidebar.vue
+++ b/frontend/src/components/Sidebar/AppSidebar.vue
@@ -192,7 +192,6 @@ import PageModal from '@/components/Modals/PageModal.vue'
 import { capture } from '@/telemetry'
 import LMSLogo from '@/components/Icons/LMSLogo.vue'
 import { useRouter } from 'vue-router'
-import InviteIcon from '@/components/Icons/InviteIcon.vue'
 import {
 	ref,
 	onMounted,
@@ -225,10 +224,10 @@ import {
 	minimize,
 	IntermediateStepModal,
 } from 'frappe-ui/frappe'
-import InviteIcon from './Icons/InviteIcon.vue'
-import UserDropdown from '@/components/UserDropdown.vue'
+import InviteIcon from '@/components/Icons/InviteIcon.vue'
+import UserDropdown from '@/components/Sidebar/UserDropdown.vue'
 import CollapseSidebar from '@/components/Icons/CollapseSidebar.vue'
-import SidebarLink from '@/components/SidebarLink.vue'
+import SidebarLink from '@/components/Sidebar/SidebarLink.vue'
 import CommandPalette from '@/components/CommandPalette/CommandPalette.vue'
 
 const { user } = sessionStore()
diff --git a/frontend/src/components/Sidebar/Apps.vue b/frontend/src/components/Sidebar/Apps.vue
index 2b44de36..74b1e62a 100644
--- a/frontend/src/components/Sidebar/Apps.vue
+++ b/frontend/src/components/Sidebar/Apps.vue
@@ -48,7 +48,7 @@ const apps = createResource({
 				name: 'frappe',
 				logo: '/assets/lms/images/desk.png',
 				title: __('Desk'),
-				route: '/app',
+				route: '/desk/lms',
 			},
 		]
 		data.map((app) => {
diff --git a/frontend/vite.config.js b/frontend/vite.config.js
index 68cbe5ec..1284a567 100644
--- a/frontend/vite.config.js
+++ b/frontend/vite.config.js
@@ -65,6 +65,6 @@ export default defineConfig(({ mode }) => ({
 			'highlight.js',
 			'plyr',
 		],
-		exclude: mode === 'production' ? [] : ['frappe-ui'],
+		//exclude: mode === 'production' ? [] : ['frappe-ui'],
 	},
 }))
diff --git a/lms/command_palette.py b/lms/command_palette.py
index 077c9697..8b7b4a87 100644
--- a/lms/command_palette.py
+++ b/lms/command_palette.py
@@ -1,4 +1,5 @@
 import frappe
+from frappe.utils import nowdate
 
 
 @frappe.whitelist()
@@ -12,14 +13,20 @@ def search_sqlite(query: str):
 	except LearningSearchIndexMissingError:
 		return []
 
+	return prepare_search_results(result)
+
+
+def prepare_search_results(result):
+	roles = frappe.get_roles()
 	groups = {}
-	print(result)
+
 	for r in result["results"]:
 		doctype = r["doctype"]
-
-		if doctype == "LMS Course":
+		if doctype == "LMS Course" and can_access_course(r, roles):
+			r["author_info"] = get_author_info(r.get("author"))
 			groups.setdefault("Courses", []).append(r)
-		elif doctype == "LMS Batch":
+		elif doctype == "LMS Batch" and can_access_batch(r, roles):
+			r["author_info"] = get_author_info(r.get("author"))
 			groups.setdefault("Batches", []).append(r)
 
 	out = []
@@ -27,3 +34,31 @@ def search_sqlite(query: str):
 		out.append({"title": key, "items": groups[key]})
 
 	return out
+
+
+def can_access_course(course, roles):
+	if can_create_course(roles):
+		return True
+	elif course.get("published"):
+		return True
+	return False
+
+
+def can_access_batch(batch, roles):
+	if can_create_batch(roles):
+		return True
+	elif batch.get("published") and batch.get("start_date") >= nowdate():
+		return True
+	return False
+
+
+def can_create_course(roles):
+	return "Course Creator" in roles or "Moderator" in roles
+
+
+def can_create_batch(roles):
+	return "Batch Evaluator" in roles or "Moderator" in roles
+
+
+def get_author_info(owner):
+	return frappe.db.get_value("User", owner, ["full_name", "user_image", "username", "email"], as_dict=True)
diff --git a/lms/sqlite.py b/lms/sqlite.py
index f8f30e03..448907a6 100644
--- a/lms/sqlite.py
+++ b/lms/sqlite.py
@@ -2,15 +2,14 @@ from contextlib import suppress
 
 import frappe
 from frappe.search.sqlite_search import SQLiteSearch, SQLiteSearchIndexMissingError
-from frappe.utils import update_progress_bar
-from redis.exceptions import ResponseError
+from frappe.utils import nowdate
 
 
 class LearningSearch(SQLiteSearch):
 	INDEX_NAME = "learning.db"
 
 	INDEX_SCHEMA = {
-		"metadata_fields": ["category", "owner", "published"],
+		"metadata_fields": ["category", "owner", "published", "published_on", "start_date"],
 		"tokenizer": "unicode61 remove_diacritics 2 tokenchars '-_'",
 	}
 
@@ -64,31 +63,6 @@ class LearningSearch(SQLiteSearch):
 		],
 	}
 
-	def can_create_course(self, roles):
-		return "Course Creator" in roles or "Moderator" in roles
-
-	def can_create_batch(self, roles):
-		return "Batch Evaluator" in roles or "Moderator" in roles
-
-	def get_records(self, doctype):
-		records = []
-		roles = frappe.get_roles()
-		filters = {}
-
-		if doctype == "LMS Course":
-			if not self.can_create_course(roles):
-				filters = {"published": 1}
-
-		if doctype == "LMS Batch":
-			if not self.can_create_batch(roles):
-				filters = {"published": 1}
-
-		records = frappe.db.get_all(doctype, filters=filters, fields=self.DOCTYPE_FIELDS[doctype])
-		for record in records:
-			record["doctype"] = doctype
-
-		return records
-
 	def build_index(self):
 		try:
 			super().build_index()
@@ -96,11 +70,21 @@ class LearningSearch(SQLiteSearch):
 			frappe.throw(e)
 
 	def get_search_filters(self):
-		roles = frappe.get_roles()
-		if not (self.can_create_course(roles) and self.can_create_batch(roles)):
-			return {"published": 1}
 		return {}
 
+	@SQLiteSearch.scoring_function
+	def get_doctype_boost(self, row, query, query_words):
+		doctype = row["doctype"]
+		if doctype == "LMS Course":
+			if row["published"]:
+				return 1.3
+		elif doctype == "LMS Batch":
+			if row["published"] and row["start_date"] >= nowdate():
+				return 1.3
+			elif row["published"]:
+				return 1.2
+		return 1.0
+
 
 class LearningSearchIndexMissingError(SQLiteSearchIndexMissingError):
 	pass

From 4c982823356c8e67aa023486715ba417637cc7a4 Mon Sep 17 00:00:00 2001
From: Jannat Patel <pateljannat2308@gmail.com>
Date: Wed, 10 Dec 2025 11:17:49 +0530
Subject: [PATCH 07/14] feat: search page

---
 frontend/src/pages/Search/Search.vue | 165 ++++++++++++++++++++++++++-
 1 file changed, 163 insertions(+), 2 deletions(-)

diff --git a/frontend/src/pages/Search/Search.vue b/frontend/src/pages/Search/Search.vue
index bc577e7b..d04c947b 100644
--- a/frontend/src/pages/Search/Search.vue
+++ b/frontend/src/pages/Search/Search.vue
@@ -4,8 +4,169 @@
 	>
 		<Breadcrumbs :items="[{ label: __('Search') }]" />
 	</header>
-	<div></div>
+	<div class="w-3/4 mx-auto py-5">
+		<div class="px-2.5">
+			<TextInput
+				ref="searchInput"
+				class="flex-1"
+				placeholder="Search for a keyword or phrase and press enter"
+				autocomplete="off"
+				:model-value="query"
+				@update:model-value="updateQuery"
+				@keydown="newSearch = true"
+				@keydown.enter="() => submit()"
+			>
+				<template #prefix>
+					<Search class="w-4 text-ink-gray-5" />
+				</template>
+				<template #suffix>
+					<div class="flex items-center">
+						<button
+							v-if="query"
+							@click="clearSearch"
+							class="p-1 size-6 grid place-content-center focus:outline-none focus:ring focus:ring-outline-gray-3 rounded"
+						>
+							<X class="w-4 text-ink-gray-7" />
+						</button>
+					</div>
+				</template>
+			</TextInput>
+			<div v-if="query" class="text-sm text-ink-gray-5 mt-2">
+				{{ searchResults.length }}
+				{{ searchResults.length === 1 ? __('match') : __('matches') }}
+			</div>
+		</div>
+
+		<div class="mt-5">
+			<div v-if="searchResults.length" class="divide-y">
+				<div
+					v-for="result in searchResults"
+					@click="navigate(result)"
+					class="flex space-x-2 hover:bg-surface-gray-2 rounded-md cursor-pointer px-2.5 py-3"
+				>
+					<Tooltip :text="result.author_info.full_name">
+						<Avatar
+							:label="result.author_info.full_name"
+							:image="result.author_info.user_image"
+							size="md"
+						/>
+					</Tooltip>
+					<div class="space-y-1 w-full">
+						<div class="flex items-center">
+							<div class="font-medium" v-html="result.title"></div>
+							<div class="text-sm text-ink-gray-5 ml-2">
+								{{ result.doctype == 'LMS Course' ? 'Course' : 'Batch' }}
+							</div>
+							<div
+								v-if="result.published_on || result.start_date"
+								class="ml-auto text-sm text-ink-gray-5"
+							>
+								{{
+									dayjs(result.published_on || result.start_date).format(
+										'DD MMM YYYY'
+									)
+								}}
+							</div>
+						</div>
+						<div class="leading-5" v-html="result.content"></div>
+					</div>
+				</div>
+			</div>
+		</div>
+	</div>
 </template>
 <script setup lang="ts">
-import { Breadcrumbs, TextInput } from 'frappe-ui'
+import {
+	Avatar,
+	Breadcrumbs,
+	createResource,
+	debounce,
+	TextInput,
+	Tooltip,
+	usePageMeta,
+} from 'frappe-ui'
+import { inject, onMounted, ref } from 'vue'
+import { Search, X } from 'lucide-vue-next'
+import { sessionStore } from '@/stores/session'
+import { useRouter } from 'vue-router'
+
+const query = ref('')
+const searchInput = ref<HTMLInputElement | null>(null)
+const newSearch = ref(false)
+const searchResults = ref<Array<any>>([])
+const { brand } = sessionStore()
+const router = useRouter()
+const dayjs = inject<any>('$dayjs')
+
+onMounted(() => {
+	if (router.currentRoute.value.query.q) {
+		query.value = router.currentRoute.value.query.q as string
+		searchInput.value.el.focus()
+		submit()
+	}
+})
+
+const updateQuery = (value: string) => {
+	query.value = value
+	router.replace({ query: value ? { q: value } : {} })
+}
+
+const submit = debounce(() => {
+	if (query.value.length > 2) {
+		search.reload()
+	}
+}, 500)
+
+const search = createResource({
+	url: 'lms.command_palette.search_sqlite',
+	makeParams: () => ({
+		query: query.value,
+	}),
+	onSuccess() {
+		generateSearchResults()
+	},
+})
+
+const generateSearchResults = () => {
+	searchResults.value = []
+	if (search.data) {
+		search.data.forEach((group: any) => {
+			group.items.forEach((item: any) => {
+				searchResults.value.push(item)
+			})
+		})
+		// sort Search results by item.score descending
+		searchResults.value.sort((a, b) => b.score - a.score)
+	}
+}
+
+const navigate = (result: any) => {
+	if (result.doctype == 'LMS Course') {
+		router.push({
+			name: 'CourseDetail',
+			params: {
+				courseName: result.name,
+			},
+		})
+	} else if (result.doctype == 'LMS Batch') {
+		router.push({
+			name: 'BatchDetail',
+			params: {
+				batchName: result.name,
+			},
+		})
+	}
+}
+
+const clearSearch = () => {
+	query.value = ''
+	searchInput.value?.focus()
+}
+
+usePageMeta(() => {
+	return {
+		title: __('Search'),
+		icon: brand.favicon,
+	}
+})
 </script>

From 820ea7e2a4f1ab7d4c7a88ce087f8fda4398e5a0 Mon Sep 17 00:00:00 2001
From: Jannat Patel <pateljannat2308@gmail.com>
Date: Thu, 11 Dec 2025 14:56:28 +0530
Subject: [PATCH 08/14] feat: search page functionality

---
 .../CommandPalette/CommandPalette.vue         |   2 +-
 .../src/components/Sidebar/AppSidebar.vue     |   9 +-
 frontend/src/pages/Search/Search.vue          | 107 ++++++++++++------
 lms/command_palette.py                        |  21 +++-
 4 files changed, 98 insertions(+), 41 deletions(-)

diff --git a/frontend/src/components/CommandPalette/CommandPalette.vue b/frontend/src/components/CommandPalette/CommandPalette.vue
index 27e6eb4b..6fd6a02d 100644
--- a/frontend/src/components/CommandPalette/CommandPalette.vue
+++ b/frontend/src/components/CommandPalette/CommandPalette.vue
@@ -215,7 +215,7 @@ const shortcutForEnter = () => {
 const navigateTo = (route: string) => {
 	show.value = false
 	query.value = ''
-	router.push({ name: route.name, params: route.params, query: route.query })
+	router.replace({ name: route.name, params: route.params, query: route.query })
 }
 
 const jumpToOptions = ref([
diff --git a/frontend/src/components/Sidebar/AppSidebar.vue b/frontend/src/components/Sidebar/AppSidebar.vue
index 39607382..b51e72a5 100644
--- a/frontend/src/components/Sidebar/AppSidebar.vue
+++ b/frontend/src/components/Sidebar/AppSidebar.vue
@@ -240,7 +240,14 @@ const showPageModal = ref(false)
 const isModerator = ref(false)
 const isInstructor = ref(false)
 const pageToEdit = ref(null)
-const { settings, sidebarSettings, activeTab, isSettingsOpen } = useSettings()
+const {
+	settings,
+	sidebarSettings,
+	activeTab,
+	isSettingsOpen,
+	isCommandPaletteOpen,
+} = useSettings()
+const settingsStore = useSettings()
 const showOnboarding = ref(false)
 const showIntermediateModal = ref(false)
 const currentStep = ref({})
diff --git a/frontend/src/pages/Search/Search.vue b/frontend/src/pages/Search/Search.vue
index d04c947b..c0d9a3ff 100644
--- a/frontend/src/pages/Search/Search.vue
+++ b/frontend/src/pages/Search/Search.vue
@@ -4,7 +4,7 @@
 	>
 		<Breadcrumbs :items="[{ label: __('Search') }]" />
 	</header>
-	<div class="w-3/4 mx-auto py-5">
+	<div class="w-4/6 mx-auto py-5">
 		<div class="px-2.5">
 			<TextInput
 				ref="searchInput"
@@ -13,7 +13,6 @@
 				autocomplete="off"
 				:model-value="query"
 				@update:model-value="updateQuery"
-				@keydown="newSearch = true"
 				@keydown.enter="() => submit()"
 			>
 				<template #prefix>
@@ -31,44 +30,63 @@
 					</div>
 				</template>
 			</TextInput>
-			<div v-if="query" class="text-sm text-ink-gray-5 mt-2">
+			<div
+				v-if="query && searchResults.length"
+				class="text-sm text-ink-gray-5 mt-2"
+			>
 				{{ searchResults.length }}
 				{{ searchResults.length === 1 ? __('match') : __('matches') }}
 			</div>
+			<div v-else-if="queryChanged" class="text-sm text-ink-gray-5 mt-2">
+				{{ __('Press enter to search') }}
+			</div>
+			<div
+				v-else-if="query && !searchResults.length"
+				class="text-sm text-ink-gray-5 mt-2"
+			>
+				{{ __('No results found') }}
+			</div>
 		</div>
 
 		<div class="mt-5">
-			<div v-if="searchResults.length" class="divide-y">
+			<div v-if="searchResults.length" class="">
 				<div
-					v-for="result in searchResults"
+					v-for="(result, index) in searchResults"
 					@click="navigate(result)"
-					class="flex space-x-2 hover:bg-surface-gray-2 rounded-md cursor-pointer px-2.5 py-3"
+					class="rounded-md cursor-pointer hover:bg-surface-gray-2 px-2"
 				>
-					<Tooltip :text="result.author_info.full_name">
-						<Avatar
-							:label="result.author_info.full_name"
-							:image="result.author_info.user_image"
-							size="md"
-						/>
-					</Tooltip>
-					<div class="space-y-1 w-full">
-						<div class="flex items-center">
-							<div class="font-medium" v-html="result.title"></div>
-							<div class="text-sm text-ink-gray-5 ml-2">
-								{{ result.doctype == 'LMS Course' ? 'Course' : 'Batch' }}
-							</div>
-							<div
-								v-if="result.published_on || result.start_date"
-								class="ml-auto text-sm text-ink-gray-5"
-							>
-								{{
-									dayjs(result.published_on || result.start_date).format(
-										'DD MMM YYYY'
-									)
-								}}
+					<div
+						class="flex space-x-2 py-3"
+						:class="{
+							'border-b': index !== searchResults.length - 1,
+						}"
+					>
+						<Tooltip :text="result.instructors_info.full_name">
+							<Avatar
+								:label="result.instructors_info.full_name"
+								:image="result.instructors_info.user_image"
+								size="md"
+							/>
+						</Tooltip>
+						<div class="space-y-1 w-full">
+							<div class="flex items-center">
+								<div class="font-medium" v-html="result.title"></div>
+								<div class="text-sm text-ink-gray-5 ml-2">
+									{{ result.doctype == 'LMS Course' ? 'Course' : 'Batch' }}
+								</div>
+								<div
+									v-if="result.published_on || result.start_date"
+									class="ml-auto text-sm text-ink-gray-5"
+								>
+									{{
+										dayjs(result.published_on || result.start_date).format(
+											'DD MMM YYYY'
+										)
+									}}
+								</div>
 							</div>
+							<div class="leading-5" v-html="result.content"></div>
 						</div>
-						<div class="leading-5" v-html="result.content"></div>
 					</div>
 				</div>
 			</div>
@@ -85,23 +103,23 @@ import {
 	Tooltip,
 	usePageMeta,
 } from 'frappe-ui'
-import { inject, onMounted, ref } from 'vue'
+import { inject, onMounted, ref, watch } from 'vue'
 import { Search, X } from 'lucide-vue-next'
 import { sessionStore } from '@/stores/session'
-import { useRouter } from 'vue-router'
+import { useRouter, useRoute } from 'vue-router'
 
 const query = ref('')
 const searchInput = ref<HTMLInputElement | null>(null)
-const newSearch = ref(false)
 const searchResults = ref<Array<any>>([])
 const { brand } = sessionStore()
 const router = useRouter()
+const route = useRoute()
+const queryChanged = ref(false)
 const dayjs = inject<any>('$dayjs')
 
 onMounted(() => {
 	if (router.currentRoute.value.query.q) {
 		query.value = router.currentRoute.value.query.q as string
-		searchInput.value.el.focus()
 		submit()
 	}
 })
@@ -130,12 +148,12 @@ const search = createResource({
 const generateSearchResults = () => {
 	searchResults.value = []
 	if (search.data) {
+		queryChanged.value = false
 		search.data.forEach((group: any) => {
 			group.items.forEach((item: any) => {
 				searchResults.value.push(item)
 			})
 		})
-		// sort Search results by item.score descending
 		searchResults.value.sort((a, b) => b.score - a.score)
 	}
 }
@@ -158,9 +176,28 @@ const navigate = (result: any) => {
 	}
 }
 
+watch(query, () => {
+	if (query.value && query.value != search.params?.query) {
+		queryChanged.value = true
+	} else if (!query.value) {
+		queryChanged.value = false
+		searchResults.value = []
+	}
+})
+
+watch(
+	() => route.query.q,
+	(newQ) => {
+		if (newQ && newQ !== query.value) {
+			query.value = newQ as string
+			submit()
+		}
+	}
+)
+
 const clearSearch = () => {
 	query.value = ''
-	searchInput.value?.focus()
+	updateQuery('')
 }
 
 usePageMeta(() => {
diff --git a/lms/command_palette.py b/lms/command_palette.py
index 8b7b4a87..fed94b3e 100644
--- a/lms/command_palette.py
+++ b/lms/command_palette.py
@@ -23,10 +23,10 @@ def prepare_search_results(result):
 	for r in result["results"]:
 		doctype = r["doctype"]
 		if doctype == "LMS Course" and can_access_course(r, roles):
-			r["author_info"] = get_author_info(r.get("author"))
+			r["instructors_info"] = get_instructor_info(doctype, r)
 			groups.setdefault("Courses", []).append(r)
 		elif doctype == "LMS Batch" and can_access_batch(r, roles):
-			r["author_info"] = get_author_info(r.get("author"))
+			r["instructors_info"] = get_instructor_info(doctype, r)
 			groups.setdefault("Batches", []).append(r)
 
 	out = []
@@ -60,5 +60,18 @@ def can_create_batch(roles):
 	return "Batch Evaluator" in roles or "Moderator" in roles
 
 
-def get_author_info(owner):
-	return frappe.db.get_value("User", owner, ["full_name", "user_image", "username", "email"], as_dict=True)
+def get_instructor_info(doctype, record):
+	instructors = frappe.get_all(
+		"Course Instructor", filters={"parenttype": doctype, "parent": record.get("name")}, pluck="instructor"
+	)
+
+	instructor = record.get("author")
+	if len(instructors):
+		instructor = instructors[0]
+
+	return frappe.db.get_value(
+		"User",
+		instructor,
+		["full_name", "email", "user_image", "username"],
+		as_dict=True,
+	)

From 819318de37000848206fd2ce24f25ccea6302b5d Mon Sep 17 00:00:00 2001
From: Jannat Patel <pateljannat2308@gmail.com>
Date: Fri, 12 Dec 2025 12:30:16 +0530
Subject: [PATCH 09/14] feat: broke down sidebar into categories

---
 .../src/components/Sidebar/AppSidebar.vue     | 167 +++------------
 frontend/src/stores/settings.js               |   9 +-
 frontend/src/utils/index.js                   | 194 +++++++++++++++---
 lms/lms/doctype/lms_course/lms_course.json    |   4 +-
 4 files changed, 203 insertions(+), 171 deletions(-)

diff --git a/frontend/src/components/Sidebar/AppSidebar.vue b/frontend/src/components/Sidebar/AppSidebar.vue
index b51e72a5..cdc8c94e 100644
--- a/frontend/src/components/Sidebar/AppSidebar.vue
+++ b/frontend/src/components/Sidebar/AppSidebar.vue
@@ -9,11 +9,21 @@
 		>
 			<UserDropdown :isCollapsed="sidebarStore.isSidebarCollapsed" />
 			<div class="flex flex-col" v-if="sidebarSettings.data">
-				<div v-for="link in sidebarLinks" class="mx-2 my-0.5">
-					<SidebarLink
-						:link="link"
-						:isCollapsed="sidebarStore.isSidebarCollapsed"
-					/>
+				<div v-for="link in sidebarLinks" class="mx-2 my-2.5">
+					<div
+						v-if="!link.hideLabel"
+						class="mb-2 mt-3 flex cursor-pointer gap-1.5 px-1 text-base font-medium text-ink-gray-5 transition-all duration-300 ease-in-out"
+					>
+						<span>{{ __(link.label) }}</span>
+					</div>
+					<nav class="space-y-1">
+						<div v-for="item in link.items">
+							<SidebarLink
+								:link="item"
+								:isCollapsed="sidebarStore.isSidebarCollapsed"
+							/>
+						</div>
+					</nav>
 				</div>
 			</div>
 			<div
@@ -235,18 +245,13 @@ const { userResource } = usersStore()
 let sidebarStore = useSidebar()
 const socket = inject('$socket')
 const unreadCount = ref(0)
-const sidebarLinks = ref(getSidebarLinks())
+const sidebarLinks = ref(null)
 const showPageModal = ref(false)
 const isModerator = ref(false)
 const isInstructor = ref(false)
 const pageToEdit = ref(null)
-const {
-	settings,
-	sidebarSettings,
-	activeTab,
-	isSettingsOpen,
-	isCommandPaletteOpen,
-} = useSettings()
+const { settings, sidebarSettings, activeTab, isSettingsOpen, programs } =
+	useSettings()
 const settingsStore = useSettings()
 const showOnboarding = ref(false)
 const showIntermediateModal = ref(false)
@@ -262,8 +267,6 @@ const iconProps = {
 }
 
 onMounted(() => {
-	addNotifications()
-	setSidebarLinks()
 	setUpOnboarding()
 	addKeyboardShortcut()
 	socket.on('publish_lms_notifications', (data) => {
@@ -278,9 +281,14 @@ const setSidebarLinks = () => {
 			onSuccess(data) {
 				Object.keys(data).forEach((key) => {
 					if (!parseInt(data[key])) {
-						sidebarLinks.value = sidebarLinks.value.filter(
+						sidebarLinks.value.forEach((link) => {
+							link.items = link.items.filter(
+								(item) => item.label.toLowerCase().split(' ').join('_') !== key
+							)
+						})
+						/* sidebarLinks.value = sidebarLinks.value?.items.filter(
 							(link) => link.label.toLowerCase().split(' ').join('_') !== key
-						)
+						) */
 					}
 				})
 			},
@@ -319,85 +327,16 @@ const unreadNotifications = createResource({
 	},
 	onSuccess(data) {
 		unreadCount.value = data
-		sidebarLinks.value = sidebarLinks.value.map((link) => {
+		/* sidebarLinks.value = sidebarLinks.value.map((link) => {
 			if (link.label === 'Notifications') {
 				link.count = data
 			}
 			return link
-		})
+		}) */
 	},
 	auto: user ? true : false,
 })
 
-const addNotifications = () => {
-	if (user) {
-		sidebarLinks.value.push({
-			label: 'Notifications',
-			icon: 'Bell',
-			to: 'Notifications',
-			activeFor: ['Notifications'],
-			count: unreadCount.value,
-		})
-	}
-}
-
-const addQuizzes = () => {
-	if (!isInstructor.value && !isModerator.value) return
-
-	const quizzesLinkExists = sidebarLinks.value.some(
-		(link) => link.label === 'Quizzes'
-	)
-	if (quizzesLinkExists) return
-
-	sidebarLinks.value.splice(4, 0, {
-		label: 'Quizzes',
-		icon: 'CircleHelp',
-		to: 'Quizzes',
-		activeFor: ['Quizzes', 'QuizForm', 'QuizSubmissionList', 'QuizSubmission'],
-	})
-}
-
-const addAssignments = () => {
-	if (!isInstructor.value && !isModerator.value) return
-
-	const assignmentsLinkExists = sidebarLinks.value.some(
-		(link) => link.label === 'Assignments'
-	)
-	if (assignmentsLinkExists) return
-
-	sidebarLinks.value.splice(5, 0, {
-		label: 'Assignments',
-		icon: 'Pencil',
-		to: 'Assignments',
-		activeFor: [
-			'Assignments',
-			'AssignmentForm',
-			'AssignmentSubmissionList',
-			'AssignmentSubmission',
-		],
-	})
-}
-
-const addProgrammingExercises = () => {
-	if (!isInstructor.value && !isModerator.value) return
-	const programmingExercisesLinkExists = sidebarLinks.value.some(
-		(link) => link.label === 'Programming Exercises'
-	)
-	if (programmingExercisesLinkExists) return
-
-	sidebarLinks.value.splice(3, 0, {
-		label: 'Programming Exercises',
-		icon: 'Code',
-		to: 'ProgrammingExercises',
-		activeFor: [
-			'ProgrammingExercises',
-			'ProgrammingExerciseForm',
-			'ProgrammingExerciseSubmissions',
-			'ProgrammingExerciseSubmission',
-		],
-	})
-}
-
 const addPrograms = async () => {
 	const programsLinkExists = sidebarLinks.value.some(
 		(link) => link.label === 'Programs'
@@ -417,45 +356,6 @@ const addPrograms = async () => {
 	})
 }
 
-const addContactUsDetails = () => {
-	if (!settings?.data?.contact_us_email && !settings?.data?.contact_us_url)
-		return
-
-	const contactUsLinkExists = sidebarLinks.value.some(
-		(link) => link.label === 'Contact Us'
-	)
-	if (contactUsLinkExists) return
-
-	sidebarLinks.value.push({
-		label: 'Contact Us',
-		icon: settings.data?.contact_us_url ? 'Headset' : 'Mail',
-		to: settings.data?.contact_us_url
-			? settings.data?.contact_us_url
-			: settings.data?.contact_us_email,
-	})
-}
-
-const checkIfCanAddProgram = async () => {
-	if (isModerator.value || isInstructor.value) {
-		return true
-	}
-	const programs = await call('lms.lms.utils.get_programs')
-	return programs.enrolled.length > 0 || programs.published.length > 0
-}
-
-const addHome = () => {
-	const homeLinkExists = sidebarLinks.value.some(
-		(link) => link.label === 'Home'
-	)
-	if (homeLinkExists) return
-	sidebarLinks.value.unshift({
-		label: 'Home',
-		icon: 'Home',
-		to: 'Home',
-		activeFor: ['Home'],
-	})
-}
-
 const openPageModal = (link) => {
 	showPageModal.value = true
 	pageToEdit.value = link
@@ -700,16 +600,15 @@ const setUpOnboarding = () => {
 	}
 }
 
-watch(userResource, () => {
-	addContactUsDetails()
+watch(userResource, async () => {
+	await userResource.promise
+	sidebarLinks.value = getSidebarLinks()
+	setSidebarLinks()
 	if (userResource.data) {
 		isModerator.value = userResource.data.is_moderator
 		isInstructor.value = userResource.data.is_instructor
-		addHome()
-		addPrograms()
-		addProgrammingExercises()
-		addQuizzes()
-		addAssignments()
+		await programs.promise
+		sidebarLinks.value = getSidebarLinks()
 		setUpOnboarding()
 	}
 })
diff --git a/frontend/src/stores/settings.js b/frontend/src/stores/settings.js
index f1689f0d..b0cf32f0 100644
--- a/frontend/src/stores/settings.js
+++ b/frontend/src/stores/settings.js
@@ -1,7 +1,6 @@
 import { defineStore } from 'pinia'
 import { ref } from 'vue'
 import { createResource } from 'frappe-ui'
-import { sessionStore } from './session'
 
 export const useSettings = defineStore('settings', () => {
 	const isSettingsOpen = ref(false)
@@ -20,10 +19,16 @@ export const useSettings = defineStore('settings', () => {
 		auto: false,
 	})
 
+	const programs = createResource({
+		url: 'lms.lms.utils.get_programs',
+		auto: false,
+	})
+
 	return {
+		activeTab,
 		isSettingsOpen,
 		isCommandPaletteOpen,
-		activeTab,
+		programs,
 		settings,
 		sidebarSettings,
 	}
diff --git a/frontend/src/utils/index.js b/frontend/src/utils/index.js
index 6b46946c..d18bc98a 100644
--- a/frontend/src/utils/index.js
+++ b/frontend/src/utils/index.js
@@ -403,51 +403,177 @@ export function getUserTimezone() {
 }
 
 export function getSidebarLinks() {
+	let links = getSidebarItems()
+
+	links.forEach((link) => {
+		link.items = link.items.filter((item) => {
+			return item.condition ? item.condition() : true
+		})
+	})
+
+	links = links.filter((link) => {
+		return link.items.length > 0
+	})
+
+	return links
+}
+
+const getSidebarItems = () => {
+	const { userResource } = usersStore()
+	const { settings } = useSettings()
+
 	return [
 		{
-			label: 'Search',
-			icon: 'Search',
-			to: 'Search',
-		},
-		{
-			label: 'Courses',
-			icon: 'BookOpen',
-			to: 'Courses',
-			activeFor: [
-				'Courses',
-				'CourseDetail',
-				'Lesson',
-				'CourseForm',
-				'LessonForm',
+			label: 'General',
+			hideLabel: true,
+			items: [
+				{
+					label: 'Home',
+					icon: 'Home',
+					to: 'Home',
+					condition: () => {
+						return userResource?.data
+					},
+				},
+				{
+					label: 'Search',
+					icon: 'Search',
+					to: 'Search',
+					condition: () => {
+						return userResource?.data
+					},
+				},
+				{
+					label: 'Notifications',
+					icon: 'Bell',
+					to: 'Notifications',
+					condition: () => {
+						return userResource?.data
+					},
+				},
 			],
 		},
 		{
-			label: 'Batches',
-			icon: 'Users',
-			to: 'Batches',
-			activeFor: ['Batches', 'BatchDetail', 'Batch', 'BatchForm'],
+			label: 'Learning',
+			hideLabel: true,
+			items: [
+				{
+					label: 'Courses',
+					icon: 'BookOpen',
+					to: 'Courses',
+					activeFor: [
+						'Courses',
+						'CourseDetail',
+						'Lesson',
+						'CourseForm',
+						'LessonForm',
+					],
+				},
+				{
+					label: 'Programs',
+					icon: 'Route',
+					to: 'Programs',
+					activeFor: ['Programs', 'ProgramDetail'],
+					await: true,
+					condition: () => {
+						return checkIfCanAddProgram()
+					},
+				},
+				{
+					label: 'Batches',
+					icon: 'Users',
+					to: 'Batches',
+					activeFor: ['Batches', 'BatchDetail', 'Batch', 'BatchForm'],
+				},
+				{
+					label: 'Certifications',
+					icon: 'GraduationCap',
+					to: 'CertifiedParticipants',
+					activeFor: ['CertifiedParticipants'],
+				},
+				{
+					label: 'Jobs',
+					icon: 'Briefcase',
+					to: 'Jobs',
+					activeFor: ['Jobs', 'JobDetail'],
+				},
+				{
+					label: 'Statistics',
+					icon: 'TrendingUp',
+					to: 'Statistics',
+					activeFor: ['Statistics'],
+				},
+				{
+					label: 'Contact Us',
+					icon: settings.data?.contact_us_url ? 'Headset' : 'Mail',
+					to: settings.data?.contact_us_url
+						? settings.data?.contact_us_url
+						: settings.data?.contact_us_email,
+					condition: () => {
+						return (
+							settings?.data?.contact_us_email ||
+							settings?.data?.contact_us_url
+						)
+					},
+				},
+			],
 		},
 		{
-			label: 'Certifications',
-			icon: 'GraduationCap',
-			to: 'CertifiedParticipants',
-			activeFor: ['CertifiedParticipants'],
-		},
-		{
-			label: 'Jobs',
-			icon: 'Briefcase',
-			to: 'Jobs',
-			activeFor: ['Jobs', 'JobDetail'],
-		},
-		{
-			label: 'Statistics',
-			icon: 'TrendingUp',
-			to: 'Statistics',
-			activeFor: ['Statistics'],
+			label: 'Assessments',
+			hideLabel: true,
+			items: [
+				{
+					label: 'Quizzes',
+					icon: 'CircleHelp',
+					to: 'Quizzes',
+					condition: () => {
+						return isAdmin()
+					},
+				},
+				{
+					label: 'Assignments',
+					icon: 'Pencil',
+					to: 'Assignments',
+					condition: () => {
+						return isAdmin()
+					},
+				},
+				{
+					label: 'Programming Exercises',
+					icon: 'Code',
+					to: 'ProgrammingExercises',
+					condition: () => {
+						return isAdmin()
+					},
+				},
+			],
 		},
 	]
 }
 
+const isAdmin = () => {
+	const { userResource } = usersStore()
+	return (
+		userResource?.data?.is_instructor ||
+		userResource?.data?.is_moderator ||
+		userResource.data?.is_evaluator
+	)
+}
+
+const checkIfCanAddProgram = () => {
+	const { userResource } = usersStore()
+	const { programs } = useSettings()
+	if (!userResource.data) return false
+	if (userResource?.data?.is_moderator || userResource?.data?.is_instructor) {
+		return true
+	}
+	console.log('programs.data', programs.data)
+	return (
+		programs.data?.enrolled.length > 0 ||
+		programs.data?.published.length > 0
+	)
+}
+
 export function getFormattedDateRange(
 	startDate,
 	endDate,
diff --git a/lms/lms/doctype/lms_course/lms_course.json b/lms/lms/doctype/lms_course/lms_course.json
index b30ed240..f4b03ffa 100644
--- a/lms/lms/doctype/lms_course/lms_course.json
+++ b/lms/lms/doctype/lms_course/lms_course.json
@@ -313,7 +313,7 @@
   }
  ],
  "make_attachments_public": 1,
- "modified": "2025-11-25 11:35:17.924569",
+ "modified": "2025-12-11 17:21:05.231761",
  "modified_by": "sayali@frappe.io",
  "module": "LMS",
  "name": "LMS Course",
@@ -336,6 +336,7 @@
    "delete": 1,
    "email": 1,
    "export": 1,
+   "import": 1,
    "print": 1,
    "read": 1,
    "report": 1,
@@ -348,6 +349,7 @@
    "delete": 1,
    "email": 1,
    "export": 1,
+   "import": 1,
    "print": 1,
    "read": 1,
    "report": 1,

From f49bb98b926975a3bb0e97fa819bc8e89d400957 Mon Sep 17 00:00:00 2001
From: Jannat Patel <pateljannat2308@gmail.com>
Date: Fri, 12 Dec 2025 16:25:45 +0530
Subject: [PATCH 10/14] fix: sidebar improvements

---
 .../CommandPalette/CommandPalette.vue         | 14 ++++--
 .../src/components/Sidebar/AppSidebar.vue     | 43 ++++++-------------
 frontend/src/utils/index.js                   |  1 -
 3 files changed, 22 insertions(+), 36 deletions(-)

diff --git a/frontend/src/components/CommandPalette/CommandPalette.vue b/frontend/src/components/CommandPalette/CommandPalette.vue
index 6fd6a02d..a32cbb00 100644
--- a/frontend/src/components/CommandPalette/CommandPalette.vue
+++ b/frontend/src/components/CommandPalette/CommandPalette.vue
@@ -10,7 +10,6 @@
 						placeholder="Search"
 						class="w-full border-none bg-transparent py-3 !pl-2 pr-4.5 text-base text-ink-gray-7 placeholder-ink-gray-4 focus:ring-0"
 						@input="onInput"
-						@keydown="onKeyDown"
 						v-model="query"
 						autocomplete="off"
 					/>
@@ -108,7 +107,7 @@ const onInput = () => {
 
 const generateSearchResults = () => {
 	search.data?.forEach((type: any) => {
-		let result = {}
+		let result: { title: string; items: any[] } = { title: '', items: [] }
 		result.title = type.title
 		type.items.forEach((item: any) => {
 			let paramName = item.doctype === 'LMS Course' ? 'courseName' : 'batchName'
@@ -126,7 +125,10 @@ const generateSearchResults = () => {
 }
 
 const appendSearchPage = () => {
-	let searchPage = {}
+	let searchPage: { title: string; items: Array<any> } = {
+		title: '',
+		items: [],
+	}
 	searchPage.title = __('Jump to')
 	searchPage.items = [
 		{
@@ -212,7 +214,11 @@ const shortcutForEnter = () => {
 	}
 }
 
-const navigateTo = (route: string) => {
+const navigateTo = (route: {
+	name: string
+	params?: Record<string, any>
+	query?: Record<string, any>
+}) => {
 	show.value = false
 	query.value = ''
 	router.replace({ name: route.name, params: route.params, query: route.query })
diff --git a/frontend/src/components/Sidebar/AppSidebar.vue b/frontend/src/components/Sidebar/AppSidebar.vue
index cdc8c94e..fbb95344 100644
--- a/frontend/src/components/Sidebar/AppSidebar.vue
+++ b/frontend/src/components/Sidebar/AppSidebar.vue
@@ -250,8 +250,7 @@ const showPageModal = ref(false)
 const isModerator = ref(false)
 const isInstructor = ref(false)
 const pageToEdit = ref(null)
-const { settings, sidebarSettings, activeTab, isSettingsOpen, programs } =
-	useSettings()
+const { sidebarSettings, activeTab, isSettingsOpen, programs } = useSettings()
 const settingsStore = useSettings()
 const showOnboarding = ref(false)
 const showIntermediateModal = ref(false)
@@ -286,9 +285,6 @@ const setSidebarLinks = () => {
 								(item) => item.label.toLowerCase().split(' ').join('_') !== key
 							)
 						})
-						/* sidebarLinks.value = sidebarLinks.value?.items.filter(
-							(link) => link.label.toLowerCase().split(' ').join('_') !== key
-						) */
 					}
 				})
 			},
@@ -327,32 +323,18 @@ const unreadNotifications = createResource({
 	},
 	onSuccess(data) {
 		unreadCount.value = data
-		/* sidebarLinks.value = sidebarLinks.value.map((link) => {
-			if (link.label === 'Notifications') {
-				link.count = data
-			}
-			return link
-		}) */
+		updateUnreadCount()
 	},
 	auto: user ? true : false,
 })
 
-const addPrograms = async () => {
-	const programsLinkExists = sidebarLinks.value.some(
-		(link) => link.label === 'Programs'
-	)
-	if (programsLinkExists) return
-
-	let canAddProgram = await checkIfCanAddProgram()
-	if (!canAddProgram) return
-	let activeFor = ['Programs', 'ProgramDetail']
-	let index = 2
-
-	sidebarLinks.value.splice(index, 0, {
-		label: 'Programs',
-		icon: 'Route',
-		to: 'Programs',
-		activeFor: activeFor,
+const updateUnreadCount = () => {
+	sidebarLinks.value?.forEach((link) => {
+		link.items.forEach((item) => {
+			if (item.label === 'Notifications') {
+				item.count = unreadCount.value || 0
+			}
+		})
 	})
 }
 
@@ -602,15 +584,14 @@ const setUpOnboarding = () => {
 
 watch(userResource, async () => {
 	await userResource.promise
-	sidebarLinks.value = getSidebarLinks()
-	setSidebarLinks()
 	if (userResource.data) {
 		isModerator.value = userResource.data.is_moderator
 		isInstructor.value = userResource.data.is_instructor
-		await programs.promise
-		sidebarLinks.value = getSidebarLinks()
+		await programs.reload()
 		setUpOnboarding()
 	}
+	sidebarLinks.value = getSidebarLinks()
+	setSidebarLinks()
 })
 
 const redirectToWebsite = () => {
diff --git a/frontend/src/utils/index.js b/frontend/src/utils/index.js
index d18bc98a..b7b1d817 100644
--- a/frontend/src/utils/index.js
+++ b/frontend/src/utils/index.js
@@ -567,7 +567,6 @@ const checkIfCanAddProgram = () => {
 	if (userResource?.data?.is_moderator || userResource?.data?.is_instructor) {
 		return true
 	}
-	console.log('programs.data', programs.data)
 	return (
 		programs.data?.enrolled.length > 0 ||
 		programs.data?.published.length > 0

From 1bc610bd76f5482d72848ad63bcdcae91f77c95c Mon Sep 17 00:00:00 2001
From: Jannat Patel <pateljannat2308@gmail.com>
Date: Fri, 12 Dec 2025 19:14:25 +0530
Subject: [PATCH 11/14] feat: search jobs from command palette

---
 frontend/src/components/CourseCard.vue |  2 +-
 frontend/src/pages/Search/Search.vue   | 39 ++++++++++++++++++++------
 lms/command_palette.py                 | 13 +++++++--
 lms/sqlite.py                          | 33 +++++++++++++++++++++-
 4 files changed, 75 insertions(+), 12 deletions(-)

diff --git a/frontend/src/components/CourseCard.vue b/frontend/src/components/CourseCard.vue
index 34f2094e..1d49ffb3 100644
--- a/frontend/src/components/CourseCard.vue
+++ b/frontend/src/components/CourseCard.vue
@@ -156,7 +156,7 @@ const getGradientColor = () => {
 		localStorage.getItem('theme') == 'light' ? 'lightMode' : 'darkMode'
 	let color = props.course.card_gradient?.toLowerCase() || 'blue'
 	let colorMap = colors[theme][color]
-	return `linear-gradient(to top right, black, ${colorMap[400]})`
+	return `linear-gradient(to top right, black, ${colorMap[200]})`
 	/* return `bg-gradient-to-br from-${color}-100 via-${color}-200 to-${color}-400` */
 	/* return `linear-gradient(to bottom right, ${colorMap[100]}, ${colorMap[400]})` */
 	/* return `radial-gradient(ellipse at 80% 20%, black 20%, ${colorMap[500]} 100%)` */
diff --git a/frontend/src/pages/Search/Search.vue b/frontend/src/pages/Search/Search.vue
index c0d9a3ff..5dbede05 100644
--- a/frontend/src/pages/Search/Search.vue
+++ b/frontend/src/pages/Search/Search.vue
@@ -61,10 +61,10 @@
 							'border-b': index !== searchResults.length - 1,
 						}"
 					>
-						<Tooltip :text="result.instructors_info.full_name">
+						<Tooltip :text="result.author_info.full_name">
 							<Avatar
-								:label="result.instructors_info.full_name"
-								:image="result.instructors_info.user_image"
+								:label="result.author_info.full_name"
+								:image="result.author_info.user_image"
 								size="md"
 							/>
 						</Tooltip>
@@ -72,16 +72,20 @@
 							<div class="flex items-center">
 								<div class="font-medium" v-html="result.title"></div>
 								<div class="text-sm text-ink-gray-5 ml-2">
-									{{ result.doctype == 'LMS Course' ? 'Course' : 'Batch' }}
+									{{ getDocTypeTitle(result.doctype) }}
 								</div>
 								<div
-									v-if="result.published_on || result.start_date"
+									v-if="
+										result.published_on || result.start_date || result.creation
+									"
 									class="ml-auto text-sm text-ink-gray-5"
 								>
 									{{
-										dayjs(result.published_on || result.start_date).format(
-											'DD MMM YYYY'
-										)
+										dayjs(
+											result.published_on ||
+												result.start_date ||
+												result.creation
+										).format('DD MMM YYYY')
 									}}
 								</div>
 							</div>
@@ -173,6 +177,13 @@ const navigate = (result: any) => {
 				batchName: result.name,
 			},
 		})
+	} else if (result.doctype == 'Job Opportunity') {
+		router.push({
+			name: 'JobDetail',
+			params: {
+				job: result.name,
+			},
+		})
 	}
 }
 
@@ -195,6 +206,18 @@ watch(
 	}
 )
 
+const getDocTypeTitle = (doctype: string) => {
+	if (doctype === 'LMS Course') {
+		return __('Course')
+	} else if (doctype === 'LMS Batch') {
+		return __('Batch')
+	} else if (doctype === 'Job Opportunity') {
+		return __('Job')
+	} else {
+		return doctype
+	}
+}
+
 const clearSearch = () => {
 	query.value = ''
 	updateQuery('')
diff --git a/lms/command_palette.py b/lms/command_palette.py
index fed94b3e..0a182cee 100644
--- a/lms/command_palette.py
+++ b/lms/command_palette.py
@@ -23,11 +23,14 @@ def prepare_search_results(result):
 	for r in result["results"]:
 		doctype = r["doctype"]
 		if doctype == "LMS Course" and can_access_course(r, roles):
-			r["instructors_info"] = get_instructor_info(doctype, r)
+			r["author_info"] = get_instructor_info(doctype, r)
 			groups.setdefault("Courses", []).append(r)
 		elif doctype == "LMS Batch" and can_access_batch(r, roles):
-			r["instructors_info"] = get_instructor_info(doctype, r)
+			r["author_info"] = get_instructor_info(doctype, r)
 			groups.setdefault("Batches", []).append(r)
+		elif doctype == "Job Opportunity" and can_access_job(r, roles):
+			r["author_info"] = get_instructor_info(doctype, r)
+			groups.setdefault("Job Opportunities", []).append(r)
 
 	out = []
 	for key in groups:
@@ -52,6 +55,12 @@ def can_access_batch(batch, roles):
 	return False
 
 
+def can_access_job(job, roles):
+	if "Moderator" in roles:
+		return True
+	return job.get("status") == "Open"
+
+
 def can_create_course(roles):
 	return "Course Creator" in roles or "Moderator" in roles
 
diff --git a/lms/sqlite.py b/lms/sqlite.py
index 448907a6..36ad7a4a 100644
--- a/lms/sqlite.py
+++ b/lms/sqlite.py
@@ -9,7 +9,15 @@ class LearningSearch(SQLiteSearch):
 	INDEX_NAME = "learning.db"
 
 	INDEX_SCHEMA = {
-		"metadata_fields": ["category", "owner", "published", "published_on", "start_date"],
+		"metadata_fields": [
+			"owner",
+			"published",
+			"published_on",
+			"start_date",
+			"status",
+			"company_name",
+			"creation",
+		],
 		"tokenizer": "unicode61 remove_diacritics 2 tokenchars '-_'",
 	}
 
@@ -38,6 +46,20 @@ class LearningSearch(SQLiteSearch):
 				{"modified": "start_date"},
 			],
 		},
+		"Job Opportunity": {
+			"fields": [
+				"name",
+				{"title": "job_title"},
+				{"content": "description"},
+				"owner",
+				"location",
+				"country",
+				"company_name",
+				"status",
+				"creation",
+				{"modified": "creation"},
+			],
+		},
 	}
 
 	DOCTYPE_FIELDS = {
@@ -61,6 +83,15 @@ class LearningSearch(SQLiteSearch):
 			"modified",
 			"owner",
 		],
+		"Job Opportunity": [
+			"name",
+			"job_title",
+			"company_name",
+			"description",
+			"creation",
+			"modified",
+			"owner",
+		],
 	}
 
 	def build_index(self):

From f783c6a62f1e91fea7f9b888a047708819e6c740 Mon Sep 17 00:00:00 2001
From: frappe-pr-bot <developers@erpnext.com>
Date: Fri, 12 Dec 2025 16:04:35 +0000
Subject: [PATCH 12/14] chore: update POT file

---
 lms/locale/main.pot | 369 ++++++++++++--------------------------------
 1 file changed, 100 insertions(+), 269 deletions(-)

diff --git a/lms/locale/main.pot b/lms/locale/main.pot
index b1ab7b81..8a8cd9a3 100644
--- a/lms/locale/main.pot
+++ b/lms/locale/main.pot
@@ -7,8 +7,8 @@ msgid ""
 msgstr ""
 "Project-Id-Version: Frappe LMS VERSION\n"
 "Report-Msgid-Bugs-To: jannat@frappe.io\n"
-"POT-Creation-Date: 2025-12-05 16:04+0000\n"
-"PO-Revision-Date: 2025-12-05 16:04+0000\n"
+"POT-Creation-Date: 2025-12-12 16:04+0000\n"
+"PO-Revision-Date: 2025-12-12 16:04+0000\n"
 "Last-Translator: jannat@frappe.io\n"
 "Language-Team: jannat@frappe.io\n"
 "MIME-Version: 1.0\n"
@@ -101,11 +101,6 @@ msgstr ""
 msgid "Acceptance for Terms and/or Policies"
 msgstr ""
 
-#. Option for the 'Status' (Select) field in DocType 'Cohort Join Request'
-#: lms/lms/doctype/cohort_join_request/cohort_join_request.json
-msgid "Accepted"
-msgstr ""
-
 #. Label of the account_id (Data) field in DocType 'LMS Zoom Settings'
 #. Label of the account_id (Data) field in DocType 'Zoom Settings'
 #: frontend/src/components/Modals/ZoomAccountModal.vue:55
@@ -195,7 +190,7 @@ msgstr ""
 msgid "Add a Student"
 msgstr ""
 
-#: frontend/src/components/Sidebar/AppSidebar.vue:616
+#: frontend/src/components/Sidebar/AppSidebar.vue:615
 msgid "Add a chapter"
 msgstr ""
 
@@ -207,7 +202,7 @@ msgstr ""
 msgid "Add a keyword and then press enter"
 msgstr ""
 
-#: frontend/src/components/Sidebar/AppSidebar.vue:617
+#: frontend/src/components/Sidebar/AppSidebar.vue:616
 msgid "Add a lesson"
 msgstr ""
 
@@ -220,7 +215,7 @@ msgstr ""
 msgid "Add a new question"
 msgstr ""
 
-#: frontend/src/components/Sidebar/AppSidebar.vue:631
+#: frontend/src/components/Sidebar/AppSidebar.vue:630
 msgid "Add a program"
 msgstr ""
 
@@ -244,7 +239,7 @@ msgstr ""
 msgid "Add at least one possible answer for this question: {0}"
 msgstr ""
 
-#: frontend/src/components/Sidebar/AppSidebar.vue:580
+#: frontend/src/components/Sidebar/AppSidebar.vue:579
 msgid "Add courses to your batch"
 msgstr ""
 
@@ -252,7 +247,7 @@ msgstr ""
 msgid "Add quiz to this video"
 msgstr ""
 
-#: frontend/src/components/Sidebar/AppSidebar.vue:559
+#: frontend/src/components/Sidebar/AppSidebar.vue:558
 msgid "Add students to your batch"
 msgstr ""
 
@@ -268,11 +263,11 @@ msgstr ""
 msgid "Add your assignment as {0}"
 msgstr ""
 
-#: frontend/src/components/Sidebar/AppSidebar.vue:492
+#: frontend/src/components/Sidebar/AppSidebar.vue:491
 msgid "Add your first chapter"
 msgstr ""
 
-#: frontend/src/components/Sidebar/AppSidebar.vue:508
+#: frontend/src/components/Sidebar/AppSidebar.vue:507
 msgid "Add your first lesson"
 msgstr ""
 
@@ -291,11 +286,7 @@ msgstr ""
 msgid "Address Line 2"
 msgstr ""
 
-#. Option for the 'Role' (Select) field in DocType 'Cohort Staff'
-#. Option for the 'Required Role' (Select) field in DocType 'Cohort Web Page'
 #. Option for the 'Role' (Select) field in DocType 'LMS Enrollment'
-#: lms/lms/doctype/cohort_staff/cohort_staff.json
-#: lms/lms/doctype/cohort_web_page/cohort_web_page.json
 #: lms/lms/doctype/lms_enrollment/lms_enrollment.json
 msgid "Admin"
 msgstr ""
@@ -541,7 +532,7 @@ msgstr ""
 #. Label of the show_assessments (Check) field in DocType 'LMS Settings'
 #: frontend/src/components/AdminBatchDashboard.vue:32
 #: frontend/src/components/Assessments.vue:5
-#: frontend/src/components/Sidebar/AppSidebar.vue:634
+#: frontend/src/components/Sidebar/AppSidebar.vue:633
 #: lms/lms/doctype/lms_settings/lms_settings.json
 #: lms/templates/assessments.html:3
 msgid "Assessments"
@@ -600,7 +591,7 @@ msgstr ""
 msgid "Assignment Title"
 msgstr ""
 
-#: frontend/src/components/Modals/AssignmentForm.vue:146
+#: frontend/src/components/Modals/AssignmentForm.vue:147
 msgid "Assignment created successfully"
 msgstr ""
 
@@ -612,7 +603,7 @@ msgstr ""
 msgid "Assignment submitted successfully"
 msgstr ""
 
-#: frontend/src/components/Modals/AssignmentForm.vue:161
+#: frontend/src/components/Modals/AssignmentForm.vue:162
 msgid "Assignment updated successfully"
 msgstr ""
 
@@ -622,8 +613,8 @@ msgid "Assignment will appear at the bottom of the lesson."
 msgstr ""
 
 #: frontend/src/components/Settings/Badges.vue:163
-#: frontend/src/components/Sidebar/AppSidebar.vue:638
-#: frontend/src/pages/Assignments.vue:208 lms/www/lms.py:272
+#: frontend/src/components/Sidebar/AppSidebar.vue:637
+#: frontend/src/pages/Assignments.vue:213 lms/www/lms.py:272
 msgid "Assignments"
 msgstr ""
 
@@ -861,7 +852,7 @@ msgstr ""
 msgid "Batch Title"
 msgstr ""
 
-#: frontend/src/pages/BatchForm.vue:594
+#: frontend/src/pages/BatchForm.vue:598
 msgid "Batch deleted successfully"
 msgstr ""
 
@@ -891,11 +882,6 @@ msgstr ""
 msgid "Batches"
 msgstr ""
 
-#. Label of the begin_date (Date) field in DocType 'Cohort'
-#: lms/lms/doctype/cohort/cohort.json
-msgid "Begin Date"
-msgstr ""
-
 #: lms/templates/emails/batch_confirmation.html:33
 #: lms/templates/emails/batch_start_reminder.html:31
 #: lms/templates/emails/certification.html:20
@@ -972,10 +958,8 @@ msgstr ""
 msgid "Cancel this evaluation?"
 msgstr ""
 
-#. Option for the 'Status' (Select) field in DocType 'Cohort'
 #. Option for the 'Stage' (Select) field in DocType 'LMS Batch Old'
 #. Option for the 'Status' (Select) field in DocType 'LMS Certificate Request'
-#: lms/lms/doctype/cohort/cohort.json
 #: lms/lms/doctype/lms_batch_old/lms_batch_old.json
 #: lms/lms/doctype/lms_certificate_request/lms_certificate_request.json
 msgid "Cancelled"
@@ -1057,10 +1041,11 @@ msgstr ""
 #. Label of the certification (Check) field in DocType 'LMS Batch'
 #. Label of the certification_section (Section Break) field in DocType 'LMS
 #. Enrollment'
+#. Label of a chart in the LMS Workspace
 #. Label of a Card Break in the LMS Workspace
 #. Label of a Link in the LMS Workspace
 #: frontend/src/components/Modals/Event.vue:411
-#: frontend/src/components/Sidebar/AppSidebar.vue:642
+#: frontend/src/components/Sidebar/AppSidebar.vue:641
 #: frontend/src/pages/BatchForm.vue:69 frontend/src/pages/Batches.vue:100
 #: frontend/src/pages/CourseCertification.vue:10
 #: frontend/src/pages/CourseCertification.vue:135
@@ -1082,7 +1067,7 @@ msgstr ""
 msgid "Certification Name"
 msgstr ""
 
-#: lms/lms/doctype/lms_certificate/lms_certificate.py:164
+#: lms/lms/doctype/lms_certificate/lms_certificate.py:163
 msgid "Certification is not enabled for this course."
 msgstr ""
 
@@ -1279,48 +1264,11 @@ msgstr ""
 msgid "Code"
 msgstr ""
 
-#. Name of a DocType
-#. Label of the cohort (Link) field in DocType 'Cohort Join Request'
-#. Label of the cohort (Link) field in DocType 'Cohort Mentor'
-#. Label of the cohort (Link) field in DocType 'Cohort Staff'
-#. Label of the cohort (Link) field in DocType 'Cohort Subgroup'
-#. Option for the 'Scope' (Select) field in DocType 'Cohort Web Page'
 #. Label of the cohort (Link) field in DocType 'LMS Enrollment'
-#: lms/lms/doctype/cohort/cohort.json
-#: lms/lms/doctype/cohort_join_request/cohort_join_request.json
-#: lms/lms/doctype/cohort_mentor/cohort_mentor.json
-#: lms/lms/doctype/cohort_staff/cohort_staff.json
-#: lms/lms/doctype/cohort_subgroup/cohort_subgroup.json
-#: lms/lms/doctype/cohort_web_page/cohort_web_page.json
 #: lms/lms/doctype/lms_enrollment/lms_enrollment.json
 msgid "Cohort"
 msgstr ""
 
-#. Name of a DocType
-#: lms/lms/doctype/cohort_join_request/cohort_join_request.json
-msgid "Cohort Join Request"
-msgstr ""
-
-#. Name of a DocType
-#: lms/lms/doctype/cohort_mentor/cohort_mentor.json
-msgid "Cohort Mentor"
-msgstr ""
-
-#. Name of a DocType
-#: lms/lms/doctype/cohort_staff/cohort_staff.json
-msgid "Cohort Staff"
-msgstr ""
-
-#. Name of a DocType
-#: lms/lms/doctype/cohort_subgroup/cohort_subgroup.json
-msgid "Cohort Subgroup"
-msgstr ""
-
-#. Name of a DocType
-#: lms/lms/doctype/cohort_web_page/cohort_web_page.json
-msgid "Cohort Web Page"
-msgstr ""
-
 #. Label of the collaboration (Select) field in DocType 'User'
 #: lms/fixtures/custom_field.json
 msgid "Collaboration Preference"
@@ -1453,10 +1401,8 @@ msgstr ""
 msgid "Complete the upcoming quiz to continue watching the video. The quiz will open in {0} {1}."
 msgstr ""
 
-#. Option for the 'Status' (Select) field in DocType 'Cohort'
 #. Option for the 'Stage' (Select) field in DocType 'LMS Batch Old'
 #. Option for the 'Status' (Select) field in DocType 'LMS Certificate Request'
-#: lms/lms/doctype/cohort/cohort.json
 #: lms/lms/doctype/lms_batch_old/lms_batch_old.json
 #: lms/lms/doctype/lms_certificate_request/lms_certificate_request.json
 #: lms/lms/widgets/CourseCard.html:78
@@ -1499,7 +1445,7 @@ msgstr ""
 msgid "Confirm Enrollment"
 msgstr ""
 
-#: frontend/src/pages/BatchForm.vue:572
+#: frontend/src/pages/BatchForm.vue:576
 msgid "Confirm your action to delete"
 msgstr ""
 
@@ -1514,7 +1460,7 @@ msgstr ""
 msgid "Confirmation Email Template"
 msgstr ""
 
-#: lms/lms/doctype/lms_certificate/lms_certificate.py:30
+#: lms/lms/doctype/lms_certificate/lms_certificate.py:29
 msgid "Congratulations on getting certified!"
 msgstr ""
 
@@ -1623,10 +1569,6 @@ msgid "Coupon(s) deleted successfully"
 msgstr ""
 
 #. Label of the course (Link) field in DocType 'Batch Course'
-#. Label of the course (Link) field in DocType 'Cohort'
-#. Label of the course (Link) field in DocType 'Cohort Mentor'
-#. Label of the course (Link) field in DocType 'Cohort Staff'
-#. Label of the course (Link) field in DocType 'Cohort Subgroup'
 #. Label of the course (Link) field in DocType 'Course Chapter'
 #. Label of the course (Link) field in DocType 'Course Lesson'
 #. Label of the course (Link) field in DocType 'Exercise Latest Submission'
@@ -1661,10 +1603,6 @@ msgstr ""
 #: frontend/src/pages/Programs/ProgramForm.vue:196
 #: frontend/src/pages/Programs/Programs.vue:35
 #: lms/lms/doctype/batch_course/batch_course.json
-#: lms/lms/doctype/cohort/cohort.json
-#: lms/lms/doctype/cohort_mentor/cohort_mentor.json
-#: lms/lms/doctype/cohort_staff/cohort_staff.json
-#: lms/lms/doctype/cohort_subgroup/cohort_subgroup.json
 #: lms/lms/doctype/course_chapter/course_chapter.json
 #: lms/lms/doctype/course_lesson/course_lesson.json
 #: lms/lms/doctype/exercise_latest_submission/exercise_latest_submission.json
@@ -1819,15 +1757,15 @@ msgstr ""
 msgid "Course already added to program"
 msgstr ""
 
-#: frontend/src/pages/CourseForm.vue:569
+#: frontend/src/pages/CourseForm.vue:572
 msgid "Course created successfully"
 msgstr ""
 
-#: frontend/src/pages/CourseForm.vue:606
+#: frontend/src/pages/CourseForm.vue:609
 msgid "Course deleted successfully"
 msgstr ""
 
-#: frontend/src/pages/CourseForm.vue:589
+#: frontend/src/pages/CourseForm.vue:592
 msgid "Course updated successfully"
 msgstr ""
 
@@ -1923,15 +1861,15 @@ msgstr ""
 msgid "Create a Quiz"
 msgstr ""
 
-#: frontend/src/components/Sidebar/AppSidebar.vue:624
+#: frontend/src/components/Sidebar/AppSidebar.vue:623
 msgid "Create a batch"
 msgstr ""
 
-#: frontend/src/components/Sidebar/AppSidebar.vue:615
+#: frontend/src/components/Sidebar/AppSidebar.vue:614
 msgid "Create a course"
 msgstr ""
 
-#: frontend/src/components/Sidebar/AppSidebar.vue:625
+#: frontend/src/components/Sidebar/AppSidebar.vue:624
 msgid "Create a live class"
 msgstr ""
 
@@ -1943,27 +1881,27 @@ msgstr ""
 msgid "Create an Assignment"
 msgstr ""
 
-#: frontend/src/components/Sidebar/AppSidebar.vue:549
+#: frontend/src/components/Sidebar/AppSidebar.vue:548
 msgid "Create your first batch"
 msgstr ""
 
-#: frontend/src/components/Sidebar/AppSidebar.vue:480
+#: frontend/src/components/Sidebar/AppSidebar.vue:479
 msgid "Create your first course"
 msgstr ""
 
-#: frontend/src/components/Sidebar/AppSidebar.vue:527
+#: frontend/src/components/Sidebar/AppSidebar.vue:526
 msgid "Create your first quiz"
 msgstr ""
 
-#: frontend/src/pages/Assignments.vue:173 frontend/src/pages/Courses.vue:355
+#: frontend/src/pages/Assignments.vue:178 frontend/src/pages/Courses.vue:355
 msgid "Created"
 msgstr ""
 
-#: frontend/src/components/Sidebar/AppSidebar.vue:621
+#: frontend/src/components/Sidebar/AppSidebar.vue:620
 msgid "Creating a batch"
 msgstr ""
 
-#: frontend/src/components/Sidebar/AppSidebar.vue:612
+#: frontend/src/components/Sidebar/AppSidebar.vue:611
 msgid "Creating a course"
 msgstr ""
 
@@ -1987,7 +1925,7 @@ msgstr ""
 msgid "Current Streak"
 msgstr ""
 
-#: frontend/src/components/Sidebar/AppSidebar.vue:648
+#: frontend/src/components/Sidebar/AppSidebar.vue:647
 msgid "Custom Certificate Templates"
 msgstr ""
 
@@ -2092,7 +2030,7 @@ msgstr ""
 #: frontend/src/components/DiscussionReplies.vue:41
 #: frontend/src/components/Settings/Badges.vue:171
 #: frontend/src/components/Settings/Coupons/CouponList.vue:133
-#: frontend/src/pages/BatchForm.vue:578 frontend/src/pages/CourseForm.vue:619
+#: frontend/src/pages/BatchForm.vue:582 frontend/src/pages/CourseForm.vue:622
 #: frontend/src/pages/ProgrammingExercises/ProgrammingExerciseForm.vue:67
 #: frontend/src/pages/Programs/ProgramForm.vue:230
 #: frontend/src/pages/Programs/ProgramForm.vue:567
@@ -2103,7 +2041,7 @@ msgstr ""
 msgid "Delete Chapter"
 msgstr ""
 
-#: frontend/src/pages/CourseForm.vue:613
+#: frontend/src/pages/CourseForm.vue:616
 msgid "Delete Course"
 msgstr ""
 
@@ -2123,11 +2061,11 @@ msgstr ""
 msgid "Delete this lesson?"
 msgstr ""
 
-#: frontend/src/pages/CourseForm.vue:614
+#: frontend/src/pages/CourseForm.vue:617
 msgid "Deleting the course will also delete all its chapters and lessons. Are you sure you want to delete this course?"
 msgstr ""
 
-#: frontend/src/pages/BatchForm.vue:573
+#: frontend/src/pages/BatchForm.vue:577
 msgid "Deleting this batch will also delete all its data including enrolled students, linked courses, assessments, feedback and discussions. Are you sure you want to continue?"
 msgstr ""
 
@@ -2142,9 +2080,6 @@ msgstr ""
 #. Label of the description (Text Editor) field in DocType 'Job Opportunity'
 #. Label of a field in the job-opportunity Web Form
 #. Label of the description (Small Text) field in DocType 'Certification'
-#. Label of the description (Markdown Editor) field in DocType 'Cohort'
-#. Label of the description (Markdown Editor) field in DocType 'Cohort
-#. Subgroup'
 #. Label of the description (Small Text) field in DocType 'LMS Badge'
 #. Label of the description (Small Text) field in DocType 'LMS Batch'
 #. Label of the description (Markdown Editor) field in DocType 'LMS Batch Old'
@@ -2158,8 +2093,6 @@ msgstr ""
 #: lms/job/doctype/job_opportunity/job_opportunity.json
 #: lms/job/web_form/job_opportunity/job_opportunity.json
 #: lms/lms/doctype/certification/certification.json
-#: lms/lms/doctype/cohort/cohort.json
-#: lms/lms/doctype/cohort_subgroup/cohort_subgroup.json
 #: lms/lms/doctype/lms_badge/lms_badge.json
 #: lms/lms/doctype/lms_batch/lms_batch.json
 #: lms/lms/doctype/lms_batch_old/lms_batch_old.json
@@ -2180,6 +2113,11 @@ msgstr ""
 msgid "Details"
 msgstr ""
 
+#. Label of the disable_pwa (Check) field in DocType 'LMS Settings'
+#: lms/lms/doctype/lms_settings/lms_settings.json
+msgid "Disable PWA"
+msgstr ""
+
 #: frontend/src/pages/CourseForm.vue:174
 msgid "Disable Self Enrollment"
 msgstr ""
@@ -2264,12 +2202,10 @@ msgstr ""
 msgid "Duplicate options found for this question."
 msgstr ""
 
-#. Label of the duration (Data) field in DocType 'Cohort'
 #. Label of the duration (Data) field in DocType 'LMS Batch Timetable'
 #. Label of the duration (Int) field in DocType 'LMS Live Class'
 #. Label of the duration (Int) field in DocType 'LMS Live Class Participant'
 #: frontend/src/components/Modals/LiveClassModal.vue:36
-#: lms/lms/doctype/cohort/cohort.json
 #: lms/lms/doctype/lms_batch_timetable/lms_batch_timetable.json
 #: lms/lms/doctype/lms_live_class/lms_live_class.json
 #: lms/lms/doctype/lms_live_class_participant/lms_live_class_participant.json
@@ -2285,16 +2221,6 @@ msgstr ""
 msgid "Duration of the live class in minutes"
 msgstr ""
 
-#. Label of the email (Link) field in DocType 'Cohort Join Request'
-#: lms/lms/doctype/cohort_join_request/cohort_join_request.json
-msgid "E-Mail"
-msgstr ""
-
-#. Label of the email (Link) field in DocType 'Cohort Mentor'
-#: lms/lms/doctype/cohort_mentor/cohort_mentor.json
-msgid "E-mail"
-msgstr ""
-
 #: frontend/src/components/BatchOverlay.vue:129
 #: frontend/src/components/CourseCardOverlay.vue:116
 #: frontend/src/components/DiscussionReplies.vue:35
@@ -2460,9 +2386,8 @@ msgstr ""
 msgid "Enabling this will publish the certificate on the certified participants page."
 msgstr ""
 
-#. Label of the end_date (Date) field in DocType 'Cohort'
 #. Label of the end_date (Date) field in DocType 'LMS Batch'
-#: lms/lms/doctype/cohort/cohort.json lms/lms/doctype/lms_batch/lms_batch.json
+#: lms/lms/doctype/lms_batch/lms_batch.json
 msgid "End Date"
 msgstr ""
 
@@ -2534,7 +2459,7 @@ msgstr ""
 msgid "Enrollment for Program {0}"
 msgstr ""
 
-#: lms/lms/utils.py:2095
+#: lms/lms/utils.py:2096
 msgid "Enrollment in this batch is restricted. Please contact the Administrator."
 msgstr ""
 
@@ -2583,7 +2508,7 @@ msgstr ""
 msgid "Error creating live class. Please try again. {0}"
 msgstr ""
 
-#: frontend/src/pages/Quizzes.vue:218
+#: frontend/src/pages/Quizzes.vue:222
 msgid "Error creating quiz: {0}"
 msgstr ""
 
@@ -2680,7 +2605,7 @@ msgstr ""
 msgid "Evaluator deleted successfully"
 msgstr ""
 
-#: lms/lms/api.py:1435
+#: lms/lms/api.py:1431
 msgid "Evaluator does not exist."
 msgstr ""
 
@@ -2829,7 +2754,7 @@ msgstr ""
 msgid "Failed to update badge assignment: "
 msgstr ""
 
-#: frontend/src/utils/index.js:685
+#: frontend/src/utils/index.js:706
 msgid "Failed to update meta tags {0}"
 msgstr ""
 
@@ -2857,10 +2782,6 @@ msgstr ""
 msgid "Field To Check"
 msgstr ""
 
-#: lms/lms/api.py:1309
-msgid "Field name is required"
-msgstr ""
-
 #. Label of the major (Data) field in DocType 'Education Detail'
 #: lms/lms/doctype/education_detail/education_detail.json
 msgid "Field of Major/Study"
@@ -3372,9 +3293,8 @@ msgstr ""
 msgid "Institution Name"
 msgstr ""
 
-#. Label of the instructor (Link) field in DocType 'Cohort'
 #. Label of the instructor (Link) field in DocType 'Course Instructor'
-#: frontend/src/pages/Home/Home.vue:149 lms/lms/doctype/cohort/cohort.json
+#: frontend/src/pages/Home/Home.vue:149
 #: lms/lms/doctype/course_instructor/course_instructor.json
 msgid "Instructor"
 msgstr ""
@@ -3410,8 +3330,8 @@ msgstr ""
 msgid "Interest"
 msgstr ""
 
-#: frontend/src/components/Sidebar/AppSidebar.vue:604
-#: frontend/src/components/Sidebar/AppSidebar.vue:607
+#: frontend/src/components/Sidebar/AppSidebar.vue:603
+#: frontend/src/components/Sidebar/AppSidebar.vue:606
 msgid "Introduction"
 msgstr ""
 
@@ -3423,17 +3343,12 @@ msgstr ""
 msgid "Invalid Quiz ID in content"
 msgstr ""
 
-#. Label of the invite_code (Data) field in DocType 'Cohort Subgroup'
-#: lms/lms/doctype/cohort_subgroup/cohort_subgroup.json
-msgid "Invite Code"
-msgstr ""
-
 #. Option for the 'Membership' (Select) field in DocType 'LMS Batch Old'
 #: lms/lms/doctype/lms_batch_old/lms_batch_old.json
 msgid "Invite Only"
 msgstr ""
 
-#: frontend/src/components/Sidebar/AppSidebar.vue:538
+#: frontend/src/components/Sidebar/AppSidebar.vue:537
 msgid "Invite your team and students"
 msgstr ""
 
@@ -3470,7 +3385,7 @@ msgstr ""
 msgid "Issue Date"
 msgstr ""
 
-#: frontend/src/components/Sidebar/AppSidebar.vue:645
+#: frontend/src/components/Sidebar/AppSidebar.vue:644
 msgid "Issue a Certificate"
 msgstr ""
 
@@ -3824,7 +3739,6 @@ msgstr ""
 #. Name of a role
 #: lms/job/doctype/job_opportunity/job_opportunity.json
 #: lms/job/doctype/lms_job_application/lms_job_application.json
-#: lms/lms/doctype/cohort_join_request/cohort_join_request.json
 #: lms/lms/doctype/course_chapter/course_chapter.json
 #: lms/lms/doctype/function/function.json
 #: lms/lms/doctype/industry/industry.json
@@ -3914,7 +3828,7 @@ msgstr ""
 msgid "Learning Consistency"
 msgstr ""
 
-#: frontend/src/components/Sidebar/AppSidebar.vue:629
+#: frontend/src/components/Sidebar/AppSidebar.vue:628
 msgid "Learning Paths"
 msgstr ""
 
@@ -4022,15 +3936,7 @@ msgstr ""
 msgid "LinkedIn ID"
 msgstr ""
 
-#. Group in Cohort's connections
-#. Group in Cohort Subgroup's connections
-#: lms/lms/doctype/cohort/cohort.json
-#: lms/lms/doctype/cohort_subgroup/cohort_subgroup.json
-msgid "Links"
-msgstr ""
-
-#. Option for the 'Status' (Select) field in DocType 'Cohort'
-#: frontend/src/pages/Courses.vue:341 lms/lms/doctype/cohort/cohort.json
+#: frontend/src/pages/Courses.vue:341
 msgid "Live"
 msgstr ""
 
@@ -4129,11 +4035,6 @@ msgstr ""
 msgid "Manage your courses and batches at a glance"
 msgstr ""
 
-#. Option for the 'Role' (Select) field in DocType 'Cohort Staff'
-#: lms/lms/doctype/cohort_staff/cohort_staff.json
-msgid "Manager"
-msgstr ""
-
 #. Option for the 'User Category' (Select) field in DocType 'User'
 #: lms/fixtures/custom_field.json lms/templates/signup-form.html:24
 msgid "Manager (Sales/Marketing/Customer)"
@@ -4183,7 +4084,7 @@ msgid "Marks to Deduct"
 msgstr ""
 
 #. Label of the max_attempts (Int) field in DocType 'LMS Quiz'
-#: frontend/src/pages/Quizzes.vue:255 lms/lms/doctype/lms_quiz/lms_quiz.json
+#: frontend/src/pages/Quizzes.vue:259 lms/lms/doctype/lms_quiz/lms_quiz.json
 msgid "Max Attempts"
 msgstr ""
 
@@ -4406,10 +4307,8 @@ msgstr ""
 msgid "Membership"
 msgstr ""
 
-#. Option for the 'Required Role' (Select) field in DocType 'Cohort Web Page'
 #. Label of the mentor (Link) field in DocType 'LMS Course Mentor Mapping'
 #. Option for the 'Member Type' (Select) field in DocType 'LMS Enrollment'
-#: lms/lms/doctype/cohort_web_page/cohort_web_page.json
 #: lms/lms/doctype/lms_course_mentor_mapping/lms_course_mentor_mapping.json
 #: lms/lms/doctype/lms_enrollment/lms_enrollment.json
 msgid "Mentor"
@@ -4476,7 +4375,7 @@ msgstr ""
 msgid "Meta Tags"
 msgstr ""
 
-#: lms/lms/api.py:1510
+#: lms/lms/api.py:1506
 msgid "Meta tags should be a list."
 msgstr ""
 
@@ -4521,7 +4420,7 @@ msgid "Moderator"
 msgstr ""
 
 #: frontend/src/pages/ProgrammingExercises/ProgrammingExerciseSubmissions.vue:286
-#: frontend/src/pages/Quizzes.vue:269
+#: frontend/src/pages/Quizzes.vue:273
 msgid "Modified"
 msgstr ""
 
@@ -4544,7 +4443,7 @@ msgstr ""
 msgid "Monday"
 msgstr ""
 
-#: frontend/src/components/Sidebar/AppSidebar.vue:653
+#: frontend/src/components/Sidebar/AppSidebar.vue:652
 msgid "Monetization"
 msgstr ""
 
@@ -4605,7 +4504,7 @@ msgstr ""
 msgid "New Coupon"
 msgstr ""
 
-#: frontend/src/pages/CourseForm.vue:700 frontend/src/pages/Courses.vue:13
+#: frontend/src/pages/CourseForm.vue:703 frontend/src/pages/Courses.vue:13
 #: lms/www/lms.py:94
 msgid "New Course"
 msgstr ""
@@ -4886,7 +4785,7 @@ msgstr ""
 msgid "Only files of type {0} will be accepted."
 msgstr ""
 
-#: frontend/src/utils/index.js:498
+#: frontend/src/utils/index.js:499
 msgid "Only image file is allowed."
 msgstr ""
 
@@ -5018,15 +4917,10 @@ msgstr ""
 msgid "PDF"
 msgstr ""
 
-#: frontend/src/components/Sidebar/AppSidebar.vue:445
+#: frontend/src/components/Sidebar/AppSidebar.vue:444
 msgid "Page deleted successfully"
 msgstr ""
 
-#. Label of the pages (Table) field in DocType 'Cohort'
-#: lms/lms/doctype/cohort/cohort.json
-msgid "Pages"
-msgstr ""
-
 #. Label of the paid_batch (Check) field in DocType 'LMS Batch'
 #: frontend/src/pages/BatchForm.vue:270
 #: lms/lms/doctype/lms_batch/lms_batch.json
@@ -5083,7 +4977,7 @@ msgstr ""
 
 #. Label of the passing_percentage (Int) field in DocType 'LMS Quiz'
 #. Label of the passing_percentage (Int) field in DocType 'LMS Quiz Submission'
-#: frontend/src/pages/QuizForm.vue:78 frontend/src/pages/Quizzes.vue:248
+#: frontend/src/pages/QuizForm.vue:78 frontend/src/pages/Quizzes.vue:252
 #: lms/lms/doctype/lms_quiz/lms_quiz.json
 #: lms/lms/doctype/lms_quiz_submission/lms_quiz_submission.json
 msgid "Passing Percentage"
@@ -5183,7 +5077,7 @@ msgstr ""
 msgid "Payment for Document Type"
 msgstr ""
 
-#: lms/lms/utils.py:2092
+#: lms/lms/utils.py:2093
 msgid "Payment is required to enroll in this batch."
 msgstr ""
 
@@ -5193,12 +5087,10 @@ msgstr ""
 msgid "Payments app is not installed"
 msgstr ""
 
-#. Option for the 'Status' (Select) field in DocType 'Cohort Join Request'
 #. Option for the 'Status' (Select) field in DocType 'LMS Certificate
 #. Evaluation'
 #. Option for the 'Status' (Select) field in DocType 'LMS Mentor Request'
 #: frontend/src/components/Modals/Event.vue:384
-#: lms/lms/doctype/cohort_join_request/cohort_join_request.json
 #: lms/lms/doctype/lms_certificate_evaluation/lms_certificate_evaluation.json
 #: lms/lms/doctype/lms_mentor_request/lms_mentor_request.json
 msgid "Pending"
@@ -5334,7 +5226,7 @@ msgstr ""
 msgid "Please login to continue with payment."
 msgstr ""
 
-#: lms/lms/utils.py:2224
+#: lms/lms/utils.py:2225
 msgid "Please login to enroll in the program."
 msgstr ""
 
@@ -5503,11 +5395,6 @@ msgstr ""
 msgid "Primary Countries"
 msgstr ""
 
-#. Label of the subgroup (Link) field in DocType 'Cohort Mentor'
-#: lms/lms/doctype/cohort_mentor/cohort_mentor.json
-msgid "Primary Subgroup"
-msgstr ""
-
 #: lms/lms/utils.py:429
 msgid "Privacy Policy"
 msgstr ""
@@ -5648,9 +5535,7 @@ msgstr ""
 msgid "Progress of students in courses and assessments"
 msgstr ""
 
-#. Option for the 'Required Role' (Select) field in DocType 'Cohort Web Page'
 #. Option for the 'Visibility' (Select) field in DocType 'LMS Batch Old'
-#: lms/lms/doctype/cohort_web_page/cohort_web_page.json
 #: lms/lms/doctype/lms_batch_old/lms_batch_old.json
 msgid "Public"
 msgstr ""
@@ -5797,7 +5682,7 @@ msgstr ""
 msgid "Quiz Title"
 msgstr ""
 
-#: frontend/src/pages/Quizzes.vue:207
+#: frontend/src/pages/Quizzes.vue:211
 msgid "Quiz created successfully"
 msgstr ""
 
@@ -5814,13 +5699,13 @@ msgstr ""
 msgid "Quiz will appear at the bottom of the lesson."
 msgstr ""
 
-#: frontend/src/components/Sidebar/AppSidebar.vue:637
-#: frontend/src/pages/QuizForm.vue:398 frontend/src/pages/Quizzes.vue:281
-#: frontend/src/pages/Quizzes.vue:291 lms/www/lms.py:250
+#: frontend/src/components/Sidebar/AppSidebar.vue:636
+#: frontend/src/pages/QuizForm.vue:398 frontend/src/pages/Quizzes.vue:285
+#: frontend/src/pages/Quizzes.vue:295 lms/www/lms.py:250
 msgid "Quizzes"
 msgstr ""
 
-#: frontend/src/pages/Quizzes.vue:229
+#: frontend/src/pages/Quizzes.vue:233
 msgid "Quizzes deleted successfully"
 msgstr ""
 
@@ -5906,9 +5791,7 @@ msgstr ""
 msgid "Registered but disabled"
 msgstr ""
 
-#. Option for the 'Status' (Select) field in DocType 'Cohort Join Request'
 #. Option for the 'Status' (Select) field in DocType 'LMS Mentor Request'
-#: lms/lms/doctype/cohort_join_request/cohort_join_request.json
 #: lms/lms/doctype/lms_mentor_request/lms_mentor_request.json
 msgid "Rejected"
 msgstr ""
@@ -5953,11 +5836,6 @@ msgstr ""
 msgid "Request for Mentorship"
 msgstr ""
 
-#. Label of the required_role (Select) field in DocType 'Cohort Web Page'
-#: lms/lms/doctype/cohort_web_page/cohort_web_page.json
-msgid "Required Role"
-msgstr ""
-
 #. Option for the 'Membership' (Select) field in DocType 'LMS Batch Old'
 #: lms/lms/doctype/lms_batch_old/lms_batch_old.json
 msgid "Restricted"
@@ -6002,9 +5880,7 @@ msgstr ""
 msgid "Reviews"
 msgstr ""
 
-#. Label of the role (Select) field in DocType 'Cohort Staff'
 #. Label of the role (Select) field in DocType 'LMS Enrollment'
-#: lms/lms/doctype/cohort_staff/cohort_staff.json
 #: lms/lms/doctype/lms_enrollment/lms_enrollment.json
 msgid "Role"
 msgstr ""
@@ -6018,7 +5894,7 @@ msgstr ""
 msgid "Role updated successfully"
 msgstr ""
 
-#: frontend/src/components/Sidebar/AppSidebar.vue:665
+#: frontend/src/components/Sidebar/AppSidebar.vue:664
 msgid "Roles"
 msgstr ""
 
@@ -6079,7 +5955,7 @@ msgstr ""
 msgid "SEO"
 msgstr ""
 
-#: frontend/src/utils/index.js:517
+#: frontend/src/utils/index.js:518
 msgid "SVG contains potentially unsafe content."
 msgstr ""
 
@@ -6136,11 +6012,6 @@ msgstr ""
 msgid "Scheduled Flow"
 msgstr ""
 
-#. Label of the scope (Select) field in DocType 'Cohort Web Page'
-#: lms/lms/doctype/cohort_web_page/cohort_web_page.json
-msgid "Scope"
-msgstr ""
-
 #. Label of the score (Int) field in DocType 'LMS Quiz Submission'
 #: frontend/src/pages/QuizSubmission.vue:39
 #: frontend/src/pages/QuizSubmissionList.vue:96
@@ -6281,16 +6152,16 @@ msgstr ""
 msgid "Set your Password"
 msgstr ""
 
-#: frontend/src/components/Sidebar/AppSidebar.vue:608
+#: frontend/src/components/Sidebar/AppSidebar.vue:607
 msgid "Setting up"
 msgstr ""
 
-#: frontend/src/components/Sidebar/AppSidebar.vue:658
+#: frontend/src/components/Sidebar/AppSidebar.vue:657
 msgid "Setting up payment gateway"
 msgstr ""
 
 #: frontend/src/components/Settings/Settings.vue:9
-#: frontend/src/components/Sidebar/AppSidebar.vue:663
+#: frontend/src/components/Sidebar/AppSidebar.vue:662
 #: frontend/src/pages/BatchForm.vue:53 frontend/src/pages/CourseForm.vue:142
 #: frontend/src/pages/ProfileRoles.vue:4
 #: frontend/src/pages/ProgrammingExercises/ProgrammingExerciseSubmission.vue:19
@@ -6322,7 +6193,7 @@ msgid "Show Answer"
 msgstr ""
 
 #. Label of the show_answers (Check) field in DocType 'LMS Quiz'
-#: frontend/src/pages/QuizForm.vue:93 frontend/src/pages/Quizzes.vue:262
+#: frontend/src/pages/QuizForm.vue:93 frontend/src/pages/Quizzes.vue:266
 #: lms/lms/doctype/lms_quiz/lms_quiz.json
 msgid "Show Answers"
 msgstr ""
@@ -6422,15 +6293,6 @@ msgstr ""
 msgid "Slot deleted successfully"
 msgstr ""
 
-#. Label of the slug (Data) field in DocType 'Cohort'
-#. Label of the slug (Data) field in DocType 'Cohort Subgroup'
-#. Label of the slug (Data) field in DocType 'Cohort Web Page'
-#: lms/lms/doctype/cohort/cohort.json
-#: lms/lms/doctype/cohort_subgroup/cohort_subgroup.json
-#: lms/lms/doctype/cohort_web_page/cohort_web_page.json
-msgid "Slug"
-msgstr ""
-
 #: frontend/src/components/BatchCard.vue:25
 #: frontend/src/components/BatchOverlay.vue:24
 msgid "Sold Out"
@@ -6455,9 +6317,7 @@ msgstr ""
 msgid "Source"
 msgstr ""
 
-#. Option for the 'Role' (Select) field in DocType 'Cohort Staff'
 #. Option for the 'Member Type' (Select) field in DocType 'LMS Enrollment'
-#: lms/lms/doctype/cohort_staff/cohort_staff.json
 #: lms/lms/doctype/lms_enrollment/lms_enrollment.json
 msgid "Staff"
 msgstr ""
@@ -6543,8 +6403,6 @@ msgstr ""
 
 #. Label of the status (Select) field in DocType 'Job Opportunity'
 #. Label of a field in the job-opportunity Web Form
-#. Label of the status (Select) field in DocType 'Cohort'
-#. Label of the status (Select) field in DocType 'Cohort Join Request'
 #. Label of the status (Select) field in DocType 'Exercise Latest Submission'
 #. Label of the status (Select) field in DocType 'Exercise Submission'
 #. Label of the status (Select) field in DocType 'LMS Assignment Submission'
@@ -6565,8 +6423,6 @@ msgstr ""
 #: frontend/src/pages/ProgrammingExercises/ProgrammingExerciseSubmissions.vue:280
 #: lms/job/doctype/job_opportunity/job_opportunity.json
 #: lms/job/web_form/job_opportunity/job_opportunity.json
-#: lms/lms/doctype/cohort/cohort.json
-#: lms/lms/doctype/cohort_join_request/cohort_join_request.json
 #: lms/lms/doctype/exercise_latest_submission/exercise_latest_submission.json
 #: lms/lms/doctype/exercise_submission/exercise_submission.json
 #: lms/lms/doctype/lms_assignment_submission/lms_assignment_submission.json
@@ -6586,11 +6442,9 @@ msgid "Status/Score"
 msgstr ""
 
 #. Option for the 'User Category' (Select) field in DocType 'User'
-#. Option for the 'Required Role' (Select) field in DocType 'Cohort Web Page'
 #. Option for the 'Member Type' (Select) field in DocType 'LMS Enrollment'
 #: frontend/src/pages/Home/Home.vue:148 frontend/src/pages/ProfileRoles.vue:38
 #: lms/fixtures/custom_field.json
-#: lms/lms/doctype/cohort_web_page/cohort_web_page.json
 #: lms/lms/doctype/lms_enrollment/lms_enrollment.json
 #: lms/templates/signup-form.html:26
 msgid "Student"
@@ -6616,11 +6470,7 @@ msgstr ""
 msgid "Students will be enrolled in a paid batch once they complete the payment"
 msgstr ""
 
-#. Label of the subgroup (Link) field in DocType 'Cohort Join Request'
-#. Option for the 'Scope' (Select) field in DocType 'Cohort Web Page'
 #. Label of the subgroup (Link) field in DocType 'LMS Enrollment'
-#: lms/lms/doctype/cohort_join_request/cohort_join_request.json
-#: lms/lms/doctype/cohort_web_page/cohort_web_page.json
 #: lms/lms/doctype/lms_enrollment/lms_enrollment.json
 msgid "Subgroup"
 msgstr ""
@@ -6706,7 +6556,7 @@ msgstr ""
 msgid "Suspicious pattern found in {0}: {1}"
 msgstr ""
 
-#: frontend/src/components/Controls/ColorSwatches.vue:50
+#: frontend/src/components/Controls/ColorSwatches.vue:52
 msgid "Swatches"
 msgstr ""
 
@@ -6714,11 +6564,6 @@ msgstr ""
 #: lms/job/doctype/job_opportunity/job_opportunity.json
 #: lms/job/doctype/job_settings/job_settings.json
 #: lms/job/doctype/lms_job_application/lms_job_application.json
-#: lms/lms/doctype/cohort/cohort.json
-#: lms/lms/doctype/cohort_join_request/cohort_join_request.json
-#: lms/lms/doctype/cohort_mentor/cohort_mentor.json
-#: lms/lms/doctype/cohort_staff/cohort_staff.json
-#: lms/lms/doctype/cohort_subgroup/cohort_subgroup.json
 #: lms/lms/doctype/course_chapter/course_chapter.json
 #: lms/lms/doctype/course_evaluator/course_evaluator.json
 #: lms/lms/doctype/course_lesson/course_lesson.json
@@ -6793,11 +6638,9 @@ msgstr ""
 msgid "Team Work"
 msgstr ""
 
-#. Label of the template (Link) field in DocType 'Cohort Web Page'
 #. Label of the template (Link) field in DocType 'LMS Certificate'
 #: frontend/src/components/Modals/BulkCertificates.vue:43
 #: frontend/src/components/Modals/Event.vue:127
-#: lms/lms/doctype/cohort_web_page/cohort_web_page.json
 #: lms/lms/doctype/lms_certificate/lms_certificate.json
 msgid "Template"
 msgstr ""
@@ -6864,7 +6707,7 @@ msgstr ""
 msgid "Thanks and Regards"
 msgstr ""
 
-#: lms/lms/utils.py:2727
+#: lms/lms/utils.py:2728
 msgid "The batch does not exist."
 msgstr ""
 
@@ -6872,7 +6715,7 @@ msgstr ""
 msgid "The batch you have enrolled for is starting tomorrow. Please be prepared and be on time for the session."
 msgstr ""
 
-#: lms/lms/utils.py:1870
+#: lms/lms/utils.py:1871
 msgid "The coupon code '{0}' is invalid."
 msgstr ""
 
@@ -6884,7 +6727,7 @@ msgstr ""
 msgid "The evaluator of this course is unavailable from {0} to {1}. Please select a date after {1}"
 msgstr ""
 
-#: lms/lms/utils.py:2711
+#: lms/lms/utils.py:2712
 msgid "The lesson does not exist."
 msgstr ""
 
@@ -6892,7 +6735,7 @@ msgstr ""
 msgid "The slot is already booked by another participant."
 msgstr ""
 
-#: lms/lms/utils.py:2067
+#: lms/lms/utils.py:2068
 msgid "The specified batch does not exist."
 msgstr ""
 
@@ -6910,7 +6753,7 @@ msgstr ""
 
 #: lms/lms/doctype/lms_batch/lms_batch.py:101
 #: lms/lms/doctype/lms_batch_enrollment/lms_batch_enrollment.py:42
-#: lms/lms/utils.py:2099
+#: lms/lms/utils.py:2100
 msgid "There are no seats available in this batch."
 msgstr ""
 
@@ -6963,15 +6806,15 @@ msgstr ""
 msgid "This class has ended"
 msgstr ""
 
-#: lms/lms/utils.py:1899
+#: lms/lms/utils.py:1900
 msgid "This coupon has expired."
 msgstr ""
 
-#: lms/lms/utils.py:1902
+#: lms/lms/utils.py:1903
 msgid "This coupon has reached its maximum usage limit."
 msgstr ""
 
-#: lms/lms/utils.py:1911
+#: lms/lms/utils.py:1912
 msgid "This coupon is not applicable to this {0}."
 msgstr ""
 
@@ -6979,7 +6822,7 @@ msgstr ""
 msgid "This course has:"
 msgstr ""
 
-#: lms/lms/utils.py:1830
+#: lms/lms/utils.py:1831
 msgid "This course is free."
 msgstr ""
 
@@ -7112,9 +6955,6 @@ msgstr ""
 msgid "Timings:"
 msgstr ""
 
-#. Label of the title (Data) field in DocType 'Cohort'
-#. Label of the title (Data) field in DocType 'Cohort Subgroup'
-#. Label of the title (Data) field in DocType 'Cohort Web Page'
 #. Label of the title (Data) field in DocType 'Course Chapter'
 #. Label of the title (Data) field in DocType 'Course Lesson'
 #. Label of the title (Data) field in DocType 'LMS Assignment'
@@ -7134,14 +6974,12 @@ msgstr ""
 #: frontend/src/components/Modals/DiscussionModal.vue:18
 #: frontend/src/components/Modals/LiveClassModal.vue:23
 #: frontend/src/components/Settings/BadgeForm.vue:19
-#: frontend/src/pages/Assignments.vue:162 frontend/src/pages/BatchForm.vue:27
+#: frontend/src/pages/Assignments.vue:167 frontend/src/pages/BatchForm.vue:27
 #: frontend/src/pages/CourseForm.vue:30 frontend/src/pages/JobForm.vue:20
 #: frontend/src/pages/ProgrammingExercises/ProgrammingExerciseForm.vue:17
 #: frontend/src/pages/Programs/ProgramForm.vue:25
 #: frontend/src/pages/QuizForm.vue:56 frontend/src/pages/Quizzes.vue:115
-#: frontend/src/pages/Quizzes.vue:235 lms/lms/doctype/cohort/cohort.json
-#: lms/lms/doctype/cohort_subgroup/cohort_subgroup.json
-#: lms/lms/doctype/cohort_web_page/cohort_web_page.json
+#: frontend/src/pages/Quizzes.vue:239
 #: lms/lms/doctype/course_chapter/course_chapter.json
 #: lms/lms/doctype/course_lesson/course_lesson.json
 #: lms/lms/doctype/lms_assignment/lms_assignment.json
@@ -7175,7 +7013,7 @@ msgstr ""
 msgid "To Date"
 msgstr ""
 
-#: lms/lms/utils.py:1844
+#: lms/lms/utils.py:1845
 msgid "To join this batch, please contact the Administrator."
 msgstr ""
 
@@ -7188,7 +7026,7 @@ msgid "Total"
 msgstr ""
 
 #. Label of the total_marks (Int) field in DocType 'LMS Quiz'
-#: frontend/src/pages/QuizForm.vue:73 frontend/src/pages/Quizzes.vue:241
+#: frontend/src/pages/QuizForm.vue:73 frontend/src/pages/Quizzes.vue:245
 #: lms/lms/doctype/lms_quiz/lms_quiz.json
 msgid "Total Marks"
 msgstr ""
@@ -7234,7 +7072,7 @@ msgstr ""
 #. Label of the type (Select) field in DocType 'LMS Quiz Question'
 #: frontend/src/components/Modals/AssessmentModal.vue:22
 #: frontend/src/components/Modals/Question.vue:44
-#: frontend/src/pages/Assignments.vue:40 frontend/src/pages/Assignments.vue:167
+#: frontend/src/pages/Assignments.vue:40 frontend/src/pages/Assignments.vue:172
 #: frontend/src/pages/JobForm.vue:25 frontend/src/pages/Jobs.vue:68
 #: frontend/src/pages/ProgrammingExercises/ProgrammingExercises.vue:53
 #: lms/job/doctype/job_opportunity/job_opportunity.json
@@ -7315,11 +7153,10 @@ msgstr ""
 msgid "Unstructured Role"
 msgstr ""
 
-#. Option for the 'Status' (Select) field in DocType 'Cohort'
 #. Option for the 'Status' (Select) field in DocType 'LMS Certificate Request'
 #. Label of the upcoming (Check) field in DocType 'LMS Course'
 #: frontend/src/pages/Batches.vue:328 frontend/src/pages/CourseForm.vue:164
-#: frontend/src/pages/Courses.vue:347 lms/lms/doctype/cohort/cohort.json
+#: frontend/src/pages/Courses.vue:347
 #: lms/lms/doctype/lms_certificate_request/lms_certificate_request.json
 #: lms/lms/doctype/lms_course/lms_course.json
 msgid "Upcoming"
@@ -7377,11 +7214,9 @@ msgid "Use HTML"
 msgstr ""
 
 #. Label of the user (Link) field in DocType 'LMS Job Application'
-#. Label of the email (Link) field in DocType 'Cohort Staff'
 #. Label of the user (Link) field in DocType 'LMS Course Interest'
 #: frontend/src/components/Settings/BadgeForm.vue:196
 #: lms/job/doctype/lms_job_application/lms_job_application.json
-#: lms/lms/doctype/cohort_staff/cohort_staff.json
 #: lms/lms/doctype/lms_course_interest/lms_course_interest.json
 msgid "User"
 msgstr ""
@@ -7637,7 +7472,7 @@ msgstr ""
 msgid "You are already enrolled for this course."
 msgstr ""
 
-#: lms/lms/utils.py:2088
+#: lms/lms/utils.py:2089
 msgid "You are already enrolled in this batch."
 msgstr ""
 
@@ -7649,11 +7484,7 @@ msgstr ""
 msgid "You are not a mentor of the course {0}"
 msgstr ""
 
-#: lms/lms/api.py:1323
-msgid "You are not allowed to access this field"
-msgstr ""
-
-#: lms/lms/doctype/lms_certificate/lms_certificate.py:161
+#: lms/lms/doctype/lms_certificate/lms_certificate.py:160
 msgid "You are not enrolled in this course."
 msgstr ""
 
@@ -7690,7 +7521,7 @@ msgstr ""
 msgid "You cannot enroll in an unpublished course."
 msgstr ""
 
-#: lms/lms/utils.py:2228
+#: lms/lms/utils.py:2229
 msgid "You cannot enroll in an unpublished program."
 msgstr ""
 
@@ -7706,11 +7537,11 @@ msgstr ""
 msgid "You cannot schedule evaluations for past slots."
 msgstr ""
 
-#: lms/lms/utils.py:2739
+#: lms/lms/utils.py:2740
 msgid "You do not have access to this batch."
 msgstr ""
 
-#: lms/lms/utils.py:2722
+#: lms/lms/utils.py:2723
 msgid "You do not have access to this course."
 msgstr ""
 
@@ -7718,7 +7549,7 @@ msgstr ""
 msgid "You do not have permission to access this page."
 msgstr ""
 
-#: lms/lms/api.py:1535 lms/lms/api.py:1539
+#: lms/lms/api.py:1531 lms/lms/api.py:1535
 msgid "You do not have permission to update meta tags."
 msgstr ""
 
@@ -7771,7 +7602,7 @@ msgstr ""
 msgid "You have got a score of {0} for the quiz {1}"
 msgstr ""
 
-#: lms/lms/doctype/lms_certificate/lms_certificate.py:170
+#: lms/lms/doctype/lms_certificate/lms_certificate.py:169
 msgid "You have not completed the course yet."
 msgstr ""
 
@@ -8127,11 +7958,11 @@ msgstr ""
 msgid "{0} is already a {1} of the course {2}"
 msgstr ""
 
-#: lms/lms/doctype/lms_certificate/lms_certificate.py:92
+#: lms/lms/doctype/lms_certificate/lms_certificate.py:91
 msgid "{0} is already certified for the batch {1}"
 msgstr ""
 
-#: lms/lms/doctype/lms_certificate/lms_certificate.py:73
+#: lms/lms/doctype/lms_certificate/lms_certificate.py:72
 msgid "{0} is already certified for the course {1}"
 msgstr ""
 

From e150225226654d905abd4b6c5db89895d4053534 Mon Sep 17 00:00:00 2001
From: Jannat Patel <31363128+pateljannat@users.noreply.github.com>
Date: Sun, 14 Dec 2025 02:05:58 +0530
Subject: [PATCH 13/14] chore: Hungarian translations

---
 lms/locale/hu.po | 26 +++++++++++++-------------
 1 file changed, 13 insertions(+), 13 deletions(-)

diff --git a/lms/locale/hu.po b/lms/locale/hu.po
index cb948fc9..745cc718 100644
--- a/lms/locale/hu.po
+++ b/lms/locale/hu.po
@@ -3,7 +3,7 @@ msgstr ""
 "Project-Id-Version: frappe\n"
 "Report-Msgid-Bugs-To: jannat@frappe.io\n"
 "POT-Creation-Date: 2025-12-05 16:04+0000\n"
-"PO-Revision-Date: 2025-12-09 18:55\n"
+"PO-Revision-Date: 2025-12-13 20:35\n"
 "Last-Translator: jannat@frappe.io\n"
 "Language-Team: Hungarian\n"
 "MIME-Version: 1.0\n"
@@ -1514,7 +1514,7 @@ msgstr ""
 #. Label of the confirmation_email_template (Link) field in DocType 'LMS Batch'
 #: lms/lms/doctype/lms_batch/lms_batch.json
 msgid "Confirmation Email Template"
-msgstr ""
+msgstr "Megerősítő E-mail Sablon"
 
 #: lms/lms/doctype/lms_certificate/lms_certificate.py:30
 msgid "Congratulations on getting certified!"
@@ -4476,7 +4476,7 @@ msgstr ""
 
 #: frontend/src/pages/BatchForm.vue:292 frontend/src/pages/CourseForm.vue:298
 msgid "Meta Tags"
-msgstr ""
+msgstr "Meta Címkék"
 
 #: lms/lms/api.py:1510
 msgid "Meta tags should be a list."
@@ -4994,7 +4994,7 @@ msgstr ""
 #. Label of the output (Data) field in DocType 'LMS Test Case Submission'
 #: lms/lms/doctype/lms_test_case_submission/lms_test_case_submission.json
 msgid "Output"
-msgstr ""
+msgstr "Kimenet"
 
 #: frontend/src/components/Settings/BadgeForm.vue:216
 #: lms/lms/doctype/lms_badge/lms_badge.js:37
@@ -5213,7 +5213,7 @@ msgstr ""
 #: lms/lms/doctype/lms_coupon/lms_coupon.json
 #: lms/lms/doctype/lms_quiz_submission/lms_quiz_submission.json
 msgid "Percentage"
-msgstr ""
+msgstr "Százalék"
 
 #. Option for the 'Grade Type' (Select) field in DocType 'Education Detail'
 #: lms/lms/doctype/education_detail/education_detail.json
@@ -5857,7 +5857,7 @@ msgstr ""
 #: lms/lms/doctype/lms_course/lms_course.json
 #: lms/lms/doctype/lms_lesson_note/lms_lesson_note.json
 msgid "Red"
-msgstr ""
+msgstr "Piros"
 
 #. Label of the redemption_count (Int) field in DocType 'LMS Coupon'
 #: frontend/src/components/Settings/Coupons/CouponList.vue:189
@@ -5873,7 +5873,7 @@ msgstr ""
 #. Timetable'
 #: lms/lms/doctype/lms_batch_timetable/lms_batch_timetable.json
 msgid "Reference DocName"
-msgstr ""
+msgstr "Hivatkozás DocNév"
 
 #. Label of the reference_doctype (Link) field in DocType 'LMS Batch Timetable'
 #. Label of the reference_doctype (Select) field in DocType 'LMS Coupon Item'
@@ -6532,7 +6532,7 @@ msgstr ""
 
 #: frontend/src/pages/Billing.vue:118
 msgid "State/Province"
-msgstr ""
+msgstr "Állam / Megye"
 
 #. Label of the tab_4_tab (Tab Break) field in DocType 'LMS Course'
 #. Label of the statistics (Check) field in DocType 'LMS Settings'
@@ -6853,7 +6853,7 @@ msgstr ""
 #: lms/lms/doctype/lms_assignment/lms_assignment.json
 #: lms/lms/doctype/lms_assignment_submission/lms_assignment_submission.json
 msgid "Text"
-msgstr ""
+msgstr "Szöveg"
 
 #: frontend/src/components/BatchFeedback.vue:6
 msgid "Thank you for providing your feedback."
@@ -7396,12 +7396,12 @@ msgstr ""
 #. Label of the user_field (Select) field in DocType 'LMS Badge'
 #: lms/lms/doctype/lms_badge/lms_badge.json
 msgid "User Field"
-msgstr ""
+msgstr "Felhasználói Mező"
 
 #. Label of the user_image (Attach Image) field in DocType 'Course Evaluator'
 #: lms/lms/doctype/course_evaluator/course_evaluator.json
 msgid "User Image"
-msgstr ""
+msgstr "Profilkép"
 
 #. Option for the 'Type' (Select) field in DocType 'LMS Question'
 #. Option for the 'Type' (Select) field in DocType 'LMS Quiz Question'
@@ -7459,7 +7459,7 @@ msgstr ""
 
 #: frontend/src/pages/Notifications.vue:39
 msgid "View"
-msgstr ""
+msgstr "Nézet"
 
 #: frontend/src/pages/JobDetail.vue:31
 msgid "View Applications"
@@ -7958,7 +7958,7 @@ msgstr ""
 
 #: frontend/src/pages/CertifiedParticipants.vue:79
 msgid "certificate"
-msgstr ""
+msgstr "tanúsítvány"
 
 #: frontend/src/pages/CertifiedParticipants.vue:78
 msgid "certificates"

From 8d41a3d688ab46340565d8e20431f76691021bea Mon Sep 17 00:00:00 2001
From: Jannat Patel <pateljannat2308@gmail.com>
Date: Mon, 15 Dec 2025 14:51:37 +0530
Subject: [PATCH 14/14] test: certificate and rating test

---
 frontend/components.d.ts                  |  5 ----
 frontend/src/components/CourseCard.vue    | 10 --------
 frontend/src/components/CourseOutline.vue |  8 +++----
 lms/desktop_icon/frappe_lms.json          | 19 +++++++++++++++
 lms/lms/test_utils.py                     | 29 +++++++++++++++++++++++
 lms/lms/utils.py                          |  1 +
 6 files changed, 53 insertions(+), 19 deletions(-)
 create mode 100644 lms/desktop_icon/frappe_lms.json

diff --git a/frontend/components.d.ts b/frontend/components.d.ts
index 95951283..2b0e68e0 100644
--- a/frontend/components.d.ts
+++ b/frontend/components.d.ts
@@ -87,11 +87,6 @@ declare module 'vue' {
     LiveClassAttendance: typeof import('./src/components/Modals/LiveClassAttendance.vue')['default']
     LiveClassModal: typeof import('./src/components/Modals/LiveClassModal.vue')['default']
     LMSLogo: typeof import('./src/components/Icons/LMSLogo.vue')['default']
-    LucideArrowDown: typeof import('~icons/lucide/arrow-down')['default']
-    LucideArrowUp: typeof import('~icons/lucide/arrow-up')['default']
-    LucideCornerDownLeft: typeof import('~icons/lucide/corner-down-left')['default']
-    LucideSearch: typeof import('~icons/lucide/search')['default']
-    LucideX: typeof import('~icons/lucide/x')['default']
     Members: typeof import('./src/components/Settings/Members.vue')['default']
     MobileLayout: typeof import('./src/components/MobileLayout.vue')['default']
     MultiSelect: typeof import('./src/components/Controls/MultiSelect.vue')['default']
diff --git a/frontend/src/components/CourseCard.vue b/frontend/src/components/CourseCard.vue
index 1d49ffb3..d0e308cf 100644
--- a/frontend/src/components/CourseCard.vue
+++ b/frontend/src/components/CourseCard.vue
@@ -157,16 +157,6 @@ const getGradientColor = () => {
 	let color = props.course.card_gradient?.toLowerCase() || 'blue'
 	let colorMap = colors[theme][color]
 	return `linear-gradient(to top right, black, ${colorMap[200]})`
-	/* return `bg-gradient-to-br from-${color}-100 via-${color}-200 to-${color}-400` */
-	/* return `linear-gradient(to bottom right, ${colorMap[100]}, ${colorMap[400]})` */
-	/* return `radial-gradient(ellipse at 80% 20%, black 20%, ${colorMap[500]} 100%)` */
-	/* return `radial-gradient(ellipse at 30% 70%, black 50%, ${colorMap[500]} 100%)` */
-	/* return `radial-gradient(ellipse at 80% 20%, ${colorMap[100]} 0%, ${colorMap[300]} 50%, ${colorMap[500]} 100%)` */
-	/* return `conic-gradient(from 180deg at 50% 50%, ${colorMap[100]} 0%, ${colorMap[200]} 50%, ${colorMap[400]} 100%)` */
-	/* return `linear-gradient(135deg, ${colorMap[100]}, ${colorMap[300]}), linear-gradient(120deg, rgba(255,255,255,0.4) 0%, transparent 60%) ` */
-	/* return `radial-gradient(circle at 20% 30%, ${colorMap[100]} 0%, transparent 40%),
-		radial-gradient(circle at 80% 40%, ${colorMap[200]} 0%, transparent 50%),
-		linear-gradient(135deg, ${colorMap[300]} 0%, ${colorMap[400]} 100%);` */
 }
 </script>
 <style>
diff --git a/frontend/src/components/CourseOutline.vue b/frontend/src/components/CourseOutline.vue
index a68b20a6..30df3aa5 100644
--- a/frontend/src/components/CourseOutline.vue
+++ b/frontend/src/components/CourseOutline.vue
@@ -95,8 +95,8 @@
 													name: allowEdit ? 'LessonForm' : 'Lesson',
 													params: {
 														courseName: courseName,
-														chapterNumber: lesson.number.split('.')[0],
-														lessonNumber: lesson.number.split('.')[1],
+														chapterNumber: lesson.number.split('-')[0],
+														lessonNumber: lesson.number.split('-')[1],
 													},
 												}"
 											>
@@ -389,8 +389,8 @@ const redirectToChapter = (chapter) => {
 
 const isActiveLesson = (lessonNumber) => {
 	return (
-		route.params.chapterNumber == lessonNumber.split('.')[0] &&
-		route.params.lessonNumber == lessonNumber.split('.')[1]
+		route.params.chapterNumber == lessonNumber.split('-')[0] &&
+		route.params.lessonNumber == lessonNumber.split('-')[1]
 	)
 }
 </script>
diff --git a/lms/desktop_icon/frappe_lms.json b/lms/desktop_icon/frappe_lms.json
new file mode 100644
index 00000000..f137321c
--- /dev/null
+++ b/lms/desktop_icon/frappe_lms.json
@@ -0,0 +1,19 @@
+{
+ "app": "lms",
+ "creation": "2025-12-15 14:31:50.704854",
+ "docstatus": 0,
+ "doctype": "Desktop Icon",
+ "hidden": 0,
+ "icon_type": "App",
+ "idx": 0,
+ "label": "Frappe LMS",
+ "link": "/lms",
+ "link_type": "External",
+ "logo_url": "/assets/lms/frontend/learning.svg",
+ "modified": "2025-12-15 14:31:50.704854",
+ "modified_by": "Administrator",
+ "name": "Frappe LMS",
+ "owner": "Administrator",
+ "roles": [],
+ "standard": 1
+}
diff --git a/lms/lms/test_utils.py b/lms/lms/test_utils.py
index 35e00d47..399c64f5 100644
--- a/lms/lms/test_utils.py
+++ b/lms/lms/test_utils.py
@@ -2,6 +2,8 @@ import unittest
 
 import frappe
 
+from lms.lms.doctype.lms_certificate.lms_certificate import get_default_certificate_template, is_certified
+
 from .utils import (
 	get_average_rating,
 	get_chapters,
@@ -39,6 +41,8 @@ class TestUtils(unittest.TestCase):
 		self.add_rating(self.course.name, self.student1.email, 0.8, "Good course")
 		self.add_rating(self.course.name, self.student2.email, 1, "Excellent course")
 
+		self.create_certificate(self.course.name, self.student1.email)
+
 	def create_a_course(self):
 		course = frappe.new_doc("LMS Course")
 		course.title = "Utility Course"
@@ -97,6 +101,15 @@ class TestUtils(unittest.TestCase):
 		else:
 			return frappe.get_doc("User", email)
 
+	def create_certificate(self, course_name, member):
+		certificate = frappe.new_doc("LMS Certificate")
+		certificate.course = course_name
+		certificate.member = member
+		certificate.issue_date = frappe.utils.nowdate()
+		certificate.template = get_default_certificate_template()
+		certificate.save()
+		return certificate
+
 	def test_simple_slugs(self):
 		self.assertEqual(slugify("hello-world"), "hello-world")
 		self.assertEqual(slugify("Hello World"), "hello-world")
@@ -205,12 +218,28 @@ class TestUtils(unittest.TestCase):
 		self.assertIsNotNone(has_student_role("student1@example.com"))
 		self.assertIsNotNone(has_student_role("student2@example.com"))
 
+	def test_is_certified(self):
+		frappe.session.user = self.student1.email
+		self.assertIsNotNone(is_certified(self.course.name))
+		frappe.session.user = self.student2.email
+		self.assertIsNone(is_certified(self.course.name))
+		frappe.session.user = "Administrator"
+
+	def test_rating_validation(self):
+		student3 = self.create_user("student3@example.com", "Emily", "Cooper", ["LMS Student"])
+		with self.assertRaises(frappe.exceptions.ValidationError):
+			self.add_rating(self.course.name, student3.email, -0.5, "Bad course")
+		frappe.session.user = "Administrator"
+		frappe.delete_doc("User", student3.email)
+
 	def tearDown(self):
 		if frappe.db.exists("LMS Course", self.course.name):
+			frappe.db.delete("LMS Certificate", {"course": self.course.name})
 			frappe.db.delete("LMS Enrollment", {"course": self.course.name})
 			frappe.db.delete("LMS Course Review", {"course": self.course.name})
 			frappe.db.delete("Course Lesson", {"course": self.course.name})
 			frappe.db.delete("Course Chapter", {"course": self.course.name})
+			frappe.db.delete("Course Instructor", {"parent": self.course.name})
 			frappe.delete_doc("LMS Course", self.course.name)
 
 		frappe.delete_doc("User", "student1@example.com")
diff --git a/lms/lms/utils.py b/lms/lms/utils.py
index e17b38da..055336ad 100644
--- a/lms/lms/utils.py
+++ b/lms/lms/utils.py
@@ -822,6 +822,7 @@ def get_course_fields():
 		"title",
 		"tags",
 		"image",
+		"video_link",
 		"card_gradient",
 		"short_introduction",
 		"published",