From bb2447e821156838a4c1ce46e2afd5f0b185b146 Mon Sep 17 00:00:00 2001
From: Jannat Patel <pateljannat2308@gmail.com>
Date: Wed, 15 Oct 2025 11:52:23 +0530
Subject: [PATCH] fix: misc issues

---
 frontend/src/components/CourseCard.vue   | 25 ++++++++++--------
 frontend/src/components/MobileLayout.vue |  8 ++++++
 frontend/src/pages/CourseForm.vue        |  5 ++++
 frontend/src/pages/Home/Home.vue         | 33 ++++++++++++------------
 lms/lms/utils.py                         |  7 +++--
 5 files changed, 47 insertions(+), 31 deletions(-)

diff --git a/frontend/src/components/CourseCard.vue b/frontend/src/components/CourseCard.vue
index 36e7c4e8..d1093cc2 100644
--- a/frontend/src/components/CourseCard.vue
+++ b/frontend/src/components/CourseCard.vue
@@ -62,7 +62,7 @@
 					<Tooltip :text="__('Enrolled Students')">
 						<span class="flex items-center">
 							<Users class="h-4 w-4 stroke-1.5 mr-1" />
-							{{ course.enrollments }}
+							{{ formatAmount(course.enrollments) }}
 						</span>
 					</Tooltip>
 				</div>
@@ -116,27 +116,30 @@
 					<CourseInstructors :instructors="course.instructors" />
 				</div>
 
-				<div v-if="course.paid_course" class="font-semibold">
-					{{ course.price }}
-				</div>
+				<div class="flex items-center space-x-2">
+					<div v-if="course.paid_course" class="font-semibold">
+						{{ course.price }}
+					</div>
 
-				<Tooltip
-					v-if="course.paid_certificate || course.enable_certification"
-					:text="__('Get Certified')"
-				>
-					<GraduationCap class="size-5 stroke-1.5 text-ink-gray-7" />
-				</Tooltip>
+					<Tooltip
+						v-if="course.paid_certificate || course.enable_certification"
+						:text="__('Get Certified')"
+					>
+						<GraduationCap class="size-5 stroke-1.5 text-ink-gray-7" />
+					</Tooltip>
+				</div>
 			</div>
 		</div>
 	</div>
 </template>
 <script setup>
 import { Award, BookOpen, GraduationCap, Star, Users } from 'lucide-vue-next'
-import UserAvatar from '@/components/UserAvatar.vue'
 import { sessionStore } from '@/stores/session'
 import { Tooltip } from 'frappe-ui'
 import { theme } from '@/utils/theme'
+import { formatAmount } from '@/utils'
 import CourseInstructors from '@/components/CourseInstructors.vue'
+import UserAvatar from '@/components/UserAvatar.vue'
 import ProgressBar from '@/components/ProgressBar.vue'
 
 const { user } = sessionStore()
diff --git a/frontend/src/components/MobileLayout.vue b/frontend/src/components/MobileLayout.vue
index 7fb29b87..b419854e 100644
--- a/frontend/src/components/MobileLayout.vue
+++ b/frontend/src/components/MobileLayout.vue
@@ -165,6 +165,14 @@ const addAssignments = () => {
 	})
 }
 
+const addProgrammingExercises = () => {
+	otherLinks.value.push({
+		label: 'Programming Exercises',
+		icon: 'Code',
+		to: 'ProgrammingExercises',
+	})
+}
+
 const addPrograms = async () => {
 	let canAddProgram = await checkIfCanAddProgram()
 	if (!canAddProgram) return
diff --git a/frontend/src/pages/CourseForm.vue b/frontend/src/pages/CourseForm.vue
index 20e0bba8..57f35800 100644
--- a/frontend/src/pages/CourseForm.vue
+++ b/frontend/src/pages/CourseForm.vue
@@ -260,12 +260,14 @@
 									v-if="course.paid_course || course.paid_certificate"
 									v-model="course.course_price"
 									:label="__('Amount')"
+									:required="course.paid_course || course.paid_certificate"
 								/>
 								<Link
 									v-if="course.paid_certificate"
 									doctype="Course Evaluator"
 									v-model="course.evaluator"
 									:label="__('Evaluator')"
+									:required="course.paid_certificate"
 									:onCreate="
 										(value, close) => openSettings('Evaluators', close)
 									"
@@ -278,11 +280,13 @@
 									v-model="course.currency"
 									:filters="{ enabled: 1 }"
 									:label="__('Currency')"
+									:required="course.paid_course || course.paid_certificate"
 								/>
 								<FormControl
 									v-if="course.paid_certificate"
 									v-model="course.timezone"
 									:label="__('Timezone')"
+									:required="course.paid_certificate"
 									:placeholder="__('e.g. IST, UTC, GMT...')"
 								/>
 							</div>
@@ -359,6 +363,7 @@ import Link from '@/components/Controls/Link.vue'
 import CourseOutline from '@/components/CourseOutline.vue'
 import MultiSelect from '@/components/Controls/MultiSelect.vue'
 import ColorSwatches from '@/components/Controls/ColorSwatches.vue'
+import { require } from 'ace-builds'
 
 const user = inject('$user')
 const newTag = ref('')
diff --git a/frontend/src/pages/Home/Home.vue b/frontend/src/pages/Home/Home.vue
index 74d40659..615709a2 100644
--- a/frontend/src/pages/Home/Home.vue
+++ b/frontend/src/pages/Home/Home.vue
@@ -5,27 +5,28 @@
 		<Breadcrumbs :items="[{ label: __('Home'), route: { name: 'Home' } }]" />
 	</header> -->
 	<div class="w-full px-5 pt-5 pb-10">
-		<div class="flex items-center justify-between">
-			<div class="space-y-2">
+		<div class="space-y-2">
+			<div class="flex items-center justify-between">
 				<div class="text-xl font-bold text-ink-gray-9">
 					{{ __('Hey') }}, {{ user.data?.full_name }} 👋
 				</div>
-				<div class="text-lg text-ink-gray-6">
-					{{ subtitle }}
+				<div>
+					<TabButtons v-if="isAdmin" v-model="currentTab" :buttons="tabs" />
+					<div
+						v-else
+						@click="showStreakModal = true"
+						class="bg-surface-amber-2 px-2 py-1 rounded-md cursor-pointer"
+					>
+						<span> 🔥 </span>
+						<span>
+							{{ streakInfo.data?.current_streak }}
+						</span>
+					</div>
 				</div>
 			</div>
-			<div>
-				<TabButtons v-if="isAdmin" v-model="currentTab" :buttons="tabs" />
-				<div
-					v-else
-					@click="showStreakModal = true"
-					class="bg-surface-amber-2 px-2 py-1 rounded-md cursor-pointer"
-				>
-					<span> 🔥 </span>
-					<span>
-						{{ streakInfo.data?.current_streak }}
-					</span>
-				</div>
+
+			<div class="text-lg text-ink-gray-6 leading-6">
+				{{ subtitle }}
 			</div>
 		</div>
 
diff --git a/lms/lms/utils.py b/lms/lms/utils.py
index c978cfcd..3acf76fb 100644
--- a/lms/lms/utils.py
+++ b/lms/lms/utils.py
@@ -2,15 +2,15 @@ import hashlib
 import json
 import re
 import string
-from datetime import datetime, timedelta
+from datetime import timedelta
 
 import frappe
-import razorpay
 import requests
 from frappe import _
 from frappe.desk.doctype.dashboard_chart.dashboard_chart import get_result
 from frappe.desk.doctype.notification_log.notification_log import make_notification_logs
 from frappe.desk.notifications import extract_mentions
+from frappe.rate_limiter import rate_limit
 from frappe.utils import (
 	add_months,
 	ceil,
@@ -18,7 +18,6 @@ from frappe.utils import (
 	cstr,
 	flt,
 	fmt_money,
-	format_date,
 	format_datetime,
 	get_datetime,
 	get_fullname,
@@ -27,7 +26,6 @@ from frappe.utils import (
 	nowtime,
 	pretty_date,
 )
-from frappe.utils.dateutils import get_period
 
 from lms.lms.md import find_macros, markdown_to_html
 
@@ -959,6 +957,7 @@ def change_currency(amount, currency, country=None):
 
 
 @frappe.whitelist(allow_guest=True)
+@rate_limit(limit=10, seconds=60 * 60)
 def get_courses(filters=None, start=0):
 	"""Returns the list of courses."""