From a710183bc7775877855e39d7bcb15755e05afdf2 Mon Sep 17 00:00:00 2001
From: Frappe PR Bot <developers@frappe.io>
Date: Wed, 14 Aug 2024 05:57:31 +0000
Subject: [PATCH 01/10] chore(release): Bumped to Version 2.2.0

---
 lms/__init__.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lms/__init__.py b/lms/__init__.py
index 9aa3f903..8a124bf6 100644
--- a/lms/__init__.py
+++ b/lms/__init__.py
@@ -1 +1 @@
-__version__ = "2.1.0"
+__version__ = "2.2.0"

From 6b12df44a0ff4f26c13643a48d3889c510a11cfd Mon Sep 17 00:00:00 2001
From: Jannat Patel <pateljannat2308@gmail.com>
Date: Wed, 14 Aug 2024 12:12:13 +0530
Subject: [PATCH 02/10] fix: settings ui

---
 frontend/src/components/Modals/Settings.vue | 116 ++++++++++----------
 frontend/src/components/SettingDetails.vue  |  16 +--
 2 files changed, 62 insertions(+), 70 deletions(-)

diff --git a/frontend/src/components/Modals/Settings.vue b/frontend/src/components/Modals/Settings.vue
index e1d8a36d..9128cd1f 100644
--- a/frontend/src/components/Modals/Settings.vue
+++ b/frontend/src/components/Modals/Settings.vue
@@ -1,5 +1,5 @@
 <template>
-	<Dialog v-model="show" :options="{ size: '6xl' }">
+	<Dialog v-model="show" :options="{ size: '3xl' }">
 		<template #body>
 			<div class="flex h-[calc(100vh_-_8rem)]">
 				<div class="flex w-52 shrink-0 flex-col bg-gray-50 p-2">
@@ -97,7 +97,7 @@ const tabs = computed(() => {
 							type: 'checkbox',
 						},
 						{
-							label: 'Apply rounding on equivalent amount',
+							label: 'Apply rounding on equivalent',
 							name: 'apply_rounding',
 							type: 'checkbox',
 						},
@@ -105,62 +105,6 @@ const tabs = computed(() => {
 				},
 			],
 		},
-		{
-			label: 'Settings',
-			hideLabel: true,
-			items: [
-				{
-					label: 'Signup',
-					icon: 'LogIn',
-					fields: [
-						{
-							label: 'Show terms of use on signup page',
-							name: 'terms_of_use',
-							type: 'checkbox',
-						},
-						{
-							label: 'Terms of Use Page',
-							name: 'terms_page',
-							type: 'Link',
-							doctype: 'Web Page',
-						},
-						{
-							label: 'Ask user category during signup',
-							name: 'user_category',
-							type: 'checkbox',
-						},
-						{
-							type: 'Column Break',
-						},
-						{
-							label: 'Show privacy policy on signup page',
-							name: 'privacy_policy',
-							type: 'checkbox',
-						},
-						{
-							label: 'Privacy Policy Page',
-							name: 'privacy_policy_page',
-							type: 'Link',
-							doctype: 'Web Page',
-						},
-						{
-							type: 'Column Break',
-						},
-						{
-							label: 'Show cookie policy on signup page',
-							name: 'cookie_policy',
-							type: 'checkbox',
-						},
-						{
-							label: 'Cookie Policy Page',
-							name: 'cookie_policy_page',
-							type: 'Link',
-							doctype: 'Web Page',
-						},
-					],
-				},
-			],
-		},
 		{
 			label: 'Settings',
 			hideLabel: true,
@@ -236,6 +180,62 @@ const tabs = computed(() => {
 				},
 			],
 		},
+		{
+			label: 'Settings',
+			hideLabel: true,
+			items: [
+				{
+					label: 'Signup',
+					icon: 'LogIn',
+					fields: [
+						{
+							label: 'Show terms of use on signup',
+							name: 'terms_of_use',
+							type: 'checkbox',
+						},
+						{
+							label: 'Terms of Use Page',
+							name: 'terms_page',
+							type: 'Link',
+							doctype: 'Web Page',
+						},
+						{
+							label: 'Show privacy policy on signup',
+							name: 'privacy_policy',
+							type: 'checkbox',
+						},
+						{
+							label: 'Privacy Policy Page',
+							name: 'privacy_policy_page',
+							type: 'Link',
+							doctype: 'Web Page',
+						},
+						
+						
+						{
+							type: 'Column Break',
+						},
+						
+						{
+							label: 'Show cookie policy on signup',
+							name: 'cookie_policy',
+							type: 'checkbox',
+						},
+						{
+							label: 'Cookie Policy Page',
+							name: 'cookie_policy_page',
+							type: 'Link',
+							doctype: 'Web Page',
+						},
+						{
+							label: 'Ask user category during signup',
+							name: 'user_category',
+							type: 'checkbox',
+						},
+					],
+				},
+			],
+		},
 		/* {
 		label: 'Settings',
 		hideLabel: true,
diff --git a/frontend/src/components/SettingDetails.vue b/frontend/src/components/SettingDetails.vue
index fe4abe9d..2fc83667 100644
--- a/frontend/src/components/SettingDetails.vue
+++ b/frontend/src/components/SettingDetails.vue
@@ -1,9 +1,9 @@
 <template>
 	<div class="flex flex-col justify-between h-full p-8">
-		<div class="flex space-x-10">
+		<div class="flex space-x-8">
 			<div v-for="(column, index) in columns" :key="index">
-				<div class="flex flex-col space-y-4">
-					<div v-for="field in column" :class="width">
+				<div class="flex flex-col space-y-4 w-60">
+					<div v-for="field in column">
 						<Link
 							v-if="field.type == 'Link'"
 							v-model="field.value"
@@ -31,11 +31,9 @@
 
 <script setup>
 import { FormControl, Button } from 'frappe-ui'
-import { computed, ref } from 'vue'
+import { computed } from 'vue'
 import Link from '@/components/Controls/Link.vue'
 
-let width = ref('w-full')
-
 const props = defineProps({
 	fields: {
 		type: Array,
@@ -71,12 +69,6 @@ const columns = computed(() => {
 		cols.push(currentColumn)
 	}
 
-	if (cols.length == 3) {
-		width.value = 'w-64'
-	} else {
-		width.value = 'w-96'
-	}
-
 	return cols
 })
 

From aeae62a45c9dfd2350dd3a91f9d9cfdafbf17ffd Mon Sep 17 00:00:00 2001
From: Jannat Patel <pateljannat2308@gmail.com>
Date: Wed, 14 Aug 2024 12:35:56 +0530
Subject: [PATCH 03/10] chore: linters

---
 frontend/src/components/Modals/Settings.vue | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/frontend/src/components/Modals/Settings.vue b/frontend/src/components/Modals/Settings.vue
index 9128cd1f..cbb53d54 100644
--- a/frontend/src/components/Modals/Settings.vue
+++ b/frontend/src/components/Modals/Settings.vue
@@ -210,12 +210,11 @@ const tabs = computed(() => {
 							type: 'Link',
 							doctype: 'Web Page',
 						},
-						
-						
+
 						{
 							type: 'Column Break',
 						},
-						
+
 						{
 							label: 'Show cookie policy on signup',
 							name: 'cookie_policy',

From ce51129e84b14cba4bc7a0b402dc047cdab2df37 Mon Sep 17 00:00:00 2001
From: Jannat Patel <pateljannat2308@gmail.com>
Date: Fri, 16 Aug 2024 11:26:11 +0530
Subject: [PATCH 04/10] feat: member list

---
 frontend/src/components/Members.vue         | 70 +++++++++++++++++++++
 frontend/src/components/Modals/Settings.vue | 25 ++++----
 frontend/src/components/SettingDetails.vue  |  2 +-
 lms/lms/api.py                              | 24 +++++++
 4 files changed, 109 insertions(+), 12 deletions(-)
 create mode 100644 frontend/src/components/Members.vue

diff --git a/frontend/src/components/Members.vue b/frontend/src/components/Members.vue
new file mode 100644
index 00000000..3a191791
--- /dev/null
+++ b/frontend/src/components/Members.vue
@@ -0,0 +1,70 @@
+<template>
+    <div class="text-base p-4">
+        <div class="font-semibold mb-1">
+            {{ __(label) }}
+        </div>
+        <div class="text-xs text-gray-600">
+            {{ __(description) }}
+        </div>
+        <div class="my-4">
+            <div v-for="member in members.data" class="grid grid-cols-5 grid-flow-row py-2">
+                <div class="flex items-center space-x-2 col-span-2">
+                    <Avatar :image="member.user_image" :label="member.full_name" size="sm" />
+                    <div>
+                       {{ member.full_name }}
+                    </div>
+                </div>
+                <div class="text-sm text-gray-700 col-span-2">
+                    {{ member.name }}
+                </div>
+                <div class="text-sm text-gray-700 justify-self-end">
+                    {{ getRole(member.role) }}
+                </div>
+            </div>
+        </div>
+        <div v-if="members.hasNextPage" class="flex justify-center">
+                <Button variant="solid" @click="members.next">
+                    <template #prefix>
+                        <component
+                            :is="icons['RefreshCw']"
+                            class="h-3 w-3 stroke-1.5"
+                        />
+                    </template>
+                    {{ __('Load More') }}
+                </Button>
+            </div>
+    </div>
+</template>
+<script setup lang="ts">
+import { createListResource, Avatar, Button, FormControl } from 'frappe-ui'
+import * as icons from 'lucide-vue-next'
+import { computed } from "vue"
+
+const props = defineProps({
+    label: {
+        type: String,
+        required: true,
+    },
+    description: {
+        type: String,
+        default: '',
+    },
+})
+
+const members = createListResource({
+    url: "lms.lms.api.get_members",
+    doctype: "User",
+    auto: true,
+})
+
+const getRole = (role) => {
+   const map = {
+       'LMS Student': 'Student',
+       'Course Creator': 'Course Instructor',
+       'Moderator': 'Moderator',
+       'Batch Evaluator': 'Evaluator'
+   }
+   return map[role]
+}
+
+</script>
\ No newline at end of file
diff --git a/frontend/src/components/Modals/Settings.vue b/frontend/src/components/Modals/Settings.vue
index e1d8a36d..522a9a76 100644
--- a/frontend/src/components/Modals/Settings.vue
+++ b/frontend/src/components/Modals/Settings.vue
@@ -28,11 +28,13 @@
 						</nav>
 					</div>
 				</div>
-				<div class="flex flex-1 flex-col overflow-y-auto">
+				<div v-if="activeTab && data.doc" class="flex flex-1 flex-col overflow-y-auto">
+					<Members v-if="activeTab.label === 'Members'" :label="activeTab.label" :description="activeTab.description"/>
 					<SettingDetails
-						v-if="activeTab && data.doc"
+						v-else
 						:fields="activeTab.fields"
 						:data="data"
+						:label="activeTab.label"
 					/>
 				</div>
 			</div>
@@ -44,6 +46,7 @@ import { Dialog, createDocumentResource } from 'frappe-ui'
 import { ref, computed, watch } from 'vue'
 import SettingDetails from '../SettingDetails.vue'
 import SidebarLink from '@/components/SidebarLink.vue'
+import Members from '@/components/Members.vue'
 
 const show = defineModel()
 const doctype = ref('LMS Settings')
@@ -236,17 +239,17 @@ const tabs = computed(() => {
 				},
 			],
 		},
-		/* {
-		label: 'Settings',
-		hideLabel: true,
-		items: [
 		{
-			label: 'Members',
-			icon: "UserRoundPlus",
-			component: markRaw(MemberSettings),
+			label: 'Settings',
+			hideLabel: true,
+			items: [
+			{
+				label: 'Members',
+				description: 'Manage the members of your learning system',
+				icon: "UserRoundPlus",
+			},
+			],
 		},
-		],
-	}, */
 	]
 
 	return _tabs.map((tab) => {
diff --git a/frontend/src/components/SettingDetails.vue b/frontend/src/components/SettingDetails.vue
index fe4abe9d..2a4d15c3 100644
--- a/frontend/src/components/SettingDetails.vue
+++ b/frontend/src/components/SettingDetails.vue
@@ -1,5 +1,5 @@
 <template>
-	<div class="flex flex-col justify-between h-full p-8">
+	<div class="flex flex-col justify-between h-full p-10">
 		<div class="flex space-x-10">
 			<div v-for="(column, index) in columns" :key="index">
 				<div class="flex flex-col space-y-4">
diff --git a/lms/lms/api.py b/lms/lms/api.py
index d04ec783..5e556fb1 100644
--- a/lms/lms/api.py
+++ b/lms/lms/api.py
@@ -562,3 +562,27 @@ def get_categories(doctype, filters):
 			categoryOptions.append({"label": category, "value": category})
 
 	return categoryOptions
+
+@frappe.whitelist()
+def get_members(start=0):
+	members = frappe.get_all("User", filters={
+		"enabled": 1,
+		"name": ["not in", ["Administrator", "Guest"]]
+	}, fields=["name", "full_name", "user_image"],
+	page_length=20, start=start)
+
+	for member in members:
+		roles = frappe.get_roles(member.name)
+		if "Moderator" in roles:
+			member.role = "Moderator"
+		elif "Course Creator" in roles:
+			member.role = "Course Creator"
+		elif "Batch Evaluator" in roles:
+			member.role = "Batch Evaluator"
+		elif "LMS Student" in roles:
+			member.role = "LMS Student"
+
+	return members
+		
+
+	
\ No newline at end of file

From 2f8acea98887881c1cbed237a9eb59fbe4046343 Mon Sep 17 00:00:00 2001
From: frappe-pr-bot <developers@erpnext.com>
Date: Fri, 16 Aug 2024 16:04:13 +0000
Subject: [PATCH 05/10] chore: update POT file

---
 lms/locale/main.pot | 112 +++++++++++++++++---------------------------
 1 file changed, 42 insertions(+), 70 deletions(-)

diff --git a/lms/locale/main.pot b/lms/locale/main.pot
index fae436ae..960fed00 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: school@frappe.io\n"
-"POT-Creation-Date: 2024-08-09 16:04+0000\n"
-"PO-Revision-Date: 2024-08-09 16:04+0000\n"
+"POT-Creation-Date: 2024-08-16 16:04+0000\n"
+"PO-Revision-Date: 2024-08-16 16:04+0000\n"
 "Last-Translator: school@frappe.io\n"
 "Language-Team: school@frappe.io\n"
 "MIME-Version: 1.0\n"
@@ -48,7 +48,7 @@ msgstr ""
 msgid "Add a Lesson"
 msgstr ""
 
-#: lms/doctype/lms_question/lms_question.py:59
+#: lms/doctype/lms_question/lms_question.py:60
 msgid "Add at least one possible answer for this question: {0}"
 msgstr ""
 
@@ -98,11 +98,6 @@ msgstr ""
 msgid "Allow accessing future dates"
 msgstr ""
 
-#. Label of the allow_student_progress (Check) field in DocType 'LMS Settings'
-#: lms/doctype/lms_settings/lms_settings.json
-msgid "Allow students to see each others progress in class"
-msgstr ""
-
 #: overrides/user.py:195
 msgid "Already Registered"
 msgstr ""
@@ -136,12 +131,6 @@ msgstr ""
 msgid "Answer"
 msgstr ""
 
-#. Option for the 'Course Creation Access Through Website To' (Select) field in
-#. DocType 'LMS Settings'
-#: lms/doctype/lms_settings/lms_settings.json
-msgid "Anyone"
-msgstr ""
-
 #. Label of the apply_gst (Check) field in DocType 'LMS Settings'
 #: lms/doctype/lms_settings/lms_settings.json
 msgid "Apply GST for India"
@@ -233,7 +222,7 @@ msgstr ""
 msgid "Assignment will appear at the bottom of the lesson."
 msgstr ""
 
-#: lms/doctype/lms_question/lms_question.py:41
+#: lms/doctype/lms_question/lms_question.py:42
 msgid "At least one option must be correct for this question."
 msgstr ""
 
@@ -850,11 +839,6 @@ msgstr ""
 msgid "Course Content"
 msgstr ""
 
-#. Label of the portal_course_creation (Select) field in DocType 'LMS Settings'
-#: lms/doctype/lms_settings/lms_settings.json
-msgid "Course Creation Access Through Website To"
-msgstr ""
-
 #. Name of a role
 #: lms/doctype/lms_course/lms_course.json
 #: lms/doctype/lms_question/lms_question.json
@@ -862,12 +846,6 @@ msgstr ""
 msgid "Course Creator"
 msgstr ""
 
-#. Option for the 'Course Creation Access Through Website To' (Select) field in
-#. DocType 'LMS Settings'
-#: lms/doctype/lms_settings/lms_settings.json
-msgid "Course Creator Role"
-msgstr ""
-
 #. Label of a Card Break in the LMS Workspace
 #: lms/workspace/lms/lms.json
 msgid "Course Data"
@@ -892,11 +870,6 @@ msgstr ""
 msgid "Course List"
 msgstr ""
 
-#. Label of the search_placeholder (Data) field in DocType 'LMS Settings'
-#: lms/doctype/lms_settings/lms_settings.json
-msgid "Course List Search Bar Placeholder"
-msgstr ""
-
 #: lms/report/course_progress_summary/course_progress_summary.py:58
 msgid "Course Name"
 msgstr ""
@@ -912,10 +885,7 @@ msgid "Course Progress Summary"
 msgstr ""
 
 #. Label of the section_break_7 (Section Break) field in DocType 'LMS Course'
-#. Label of the course_settings_section (Section Break) field in DocType 'LMS
-#. Settings'
 #: lms/doctype/lms_course/lms_course.json
-#: lms/doctype/lms_settings/lms_settings.json
 msgid "Course Settings"
 msgstr ""
 
@@ -1116,7 +1086,7 @@ msgstr ""
 msgid "Dream Companies"
 msgstr ""
 
-#: lms/doctype/lms_question/lms_question.py:31
+#: lms/doctype/lms_question/lms_question.py:32
 msgid "Duplicate options found for this question."
 msgstr ""
 
@@ -1254,7 +1224,7 @@ msgstr ""
 msgid "Enter the correct answer"
 msgstr ""
 
-#: lms/utils.py:1088
+#: lms/utils.py:1081
 msgid "Error during payment: {0} Please contact the Administrator. Amount {1} Currency {2} Formatted {3}"
 msgstr ""
 
@@ -1539,6 +1509,7 @@ msgid "Here are a few courses we recommend for you to get started with {0}"
 msgstr ""
 
 #: lms/notification/certificate_request_creation/certificate_request_creation.html:6
+#: templates/emails/certificate_request_notification.html:1
 msgid "Hey {0}"
 msgstr ""
 
@@ -1738,7 +1709,7 @@ msgstr ""
 msgid "Invalid Start or End Time."
 msgstr ""
 
-#: lms/utils.py:932
+#: lms/utils.py:925
 msgid "Invalid document provided."
 msgstr ""
 
@@ -2328,7 +2299,6 @@ msgstr ""
 
 #. Label of the mentor_request_section (Section Break) field in DocType 'LMS
 #. Settings'
-#. Label of the mentor_request_tab (Tab Break) field in DocType 'LMS Settings'
 #: lms/doctype/lms_settings/lms_settings.json
 msgid "Mentor Request"
 msgstr ""
@@ -2385,11 +2355,11 @@ msgstr ""
 msgid "Modified By"
 msgstr ""
 
-#: lms/api.py:188
+#: lms/api.py:189
 msgid "Module Name is incorrect or does not exist."
 msgstr ""
 
-#: lms/api.py:184
+#: lms/api.py:185
 msgid "Module is incorrect."
 msgstr ""
 
@@ -2430,11 +2400,11 @@ msgstr ""
 msgid "New Sign Up"
 msgstr ""
 
-#: lms/utils.py:619
+#: lms/utils.py:612
 msgid "New comment in batch {0}"
 msgstr ""
 
-#: lms/utils.py:612
+#: lms/utils.py:605
 msgid "New reply on the topic {0} in course {1}"
 msgstr ""
 
@@ -2471,11 +2441,6 @@ msgstr ""
 msgid "No courses under review"
 msgstr ""
 
-#: templates/search_course/search_course.html:61
-#: templates/search_course/search_course.js:47
-msgid "No result found"
-msgstr ""
-
 #: templates/course_list.html:13
 msgid "No {0}"
 msgstr ""
@@ -2719,7 +2684,7 @@ msgstr ""
 msgid "Payment for Document Type"
 msgstr ""
 
-#: lms/utils.py:949
+#: lms/utils.py:942
 msgid "Payment for {0} course"
 msgstr ""
 
@@ -2778,12 +2743,13 @@ msgstr ""
 msgid "Please enter your answer"
 msgstr ""
 
-#: lms/api.py:180
+#: lms/api.py:181
 msgid "Please login to continue with payment."
 msgstr ""
 
 #: lms/notification/certificate_request_creation/certificate_request_creation.html:9
 #: lms/notification/certificate_request_reminder/certificate_request_reminder.html:8
+#: templates/emails/certificate_request_notification.html:4
 msgid "Please prepare well and be on time for the evaluations."
 msgstr ""
 
@@ -2952,6 +2918,11 @@ msgstr ""
 msgid "Question "
 msgstr ""
 
+#. Label of the question_detail (Text) field in DocType 'LMS Quiz Question'
+#: lms/doctype/lms_quiz_question/lms_quiz_question.json
+msgid "Question Detail"
+msgstr ""
+
 #. Label of the question_name (Link) field in DocType 'LMS Quiz Result'
 #: lms/doctype/lms_quiz_result/lms_quiz_result.json
 msgid "Question Name"
@@ -3214,11 +3185,6 @@ msgstr ""
 msgid "Set your Password"
 msgstr ""
 
-#. Label of the section_break_hsiv (Section Break) field in DocType 'LMS Quiz'
-#: lms/doctype/lms_quiz/lms_quiz.json
-msgid "Settings"
-msgstr ""
-
 #. Label of the short_introduction (Small Text) field in DocType 'LMS Course'
 #: lms/doctype/lms_course/lms_course.json
 msgid "Short Introduction"
@@ -3670,7 +3636,7 @@ msgstr ""
 msgid "The course {0} is now available on {1}."
 msgstr ""
 
-#: lms/doctype/lms_certificate_request/lms_certificate_request.py:44
+#: lms/doctype/lms_certificate_request/lms_certificate_request.py:47
 msgid "The evaluator of this course is unavailable from {0} to {1}. Please select a date after {1}"
 msgstr ""
 
@@ -3678,7 +3644,7 @@ msgstr ""
 msgid "The quiz has a time limit. For each question you will be given {0} seconds."
 msgstr ""
 
-#: lms/doctype/lms_certificate_request/lms_certificate_request.py:62
+#: lms/doctype/lms_certificate_request/lms_certificate_request.py:65
 msgid "The slot is already booked by another participant."
 msgstr ""
 
@@ -3694,7 +3660,7 @@ msgstr ""
 msgid "There are no {0} on this site."
 msgstr ""
 
-#: lms/utils.py:1070
+#: lms/utils.py:1063
 msgid "There is a problem with the payment gateway. Please contact the Administrator to proceed."
 msgstr ""
 
@@ -3709,7 +3675,7 @@ msgstr ""
 msgid "This certificate does no expire"
 msgstr ""
 
-#: lms/utils.py:1028 lms/utils.py:1769
+#: lms/utils.py:1021 lms/utils.py:1762
 msgid "This course is free."
 msgstr ""
 
@@ -3823,7 +3789,7 @@ msgstr ""
 msgid "To Date is mandatory in Work Experience."
 msgstr ""
 
-#: lms/utils.py:1037 lms/utils.py:1780
+#: lms/utils.py:1030 lms/utils.py:1773
 msgid "To join this batch, please contact the Administrator."
 msgstr ""
 
@@ -4077,15 +4043,15 @@ msgstr ""
 msgid "Write a review"
 msgstr ""
 
-#: lms/doctype/lms_certificate_request/lms_certificate_request.py:86
+#: lms/doctype/lms_certificate_request/lms_certificate_request.py:89
 msgid "You already have an evaluation on {0} at {1} for the course {2}."
 msgstr ""
 
-#: lms/api.py:204
+#: lms/api.py:205
 msgid "You are already enrolled for this batch."
 msgstr ""
 
-#: lms/api.py:196
+#: lms/api.py:197
 msgid "You are already enrolled for this course."
 msgstr ""
 
@@ -4110,11 +4076,11 @@ msgstr ""
 msgid "You can find their resume attached to this email."
 msgstr ""
 
-#: lms/doctype/lms_certificate_request/lms_certificate_request.py:106
+#: lms/doctype/lms_certificate_request/lms_certificate_request.py:109
 msgid "You cannot schedule evaluations after {0}."
 msgstr ""
 
-#: lms/doctype/lms_certificate_request/lms_certificate_request.py:95
+#: lms/doctype/lms_certificate_request/lms_certificate_request.py:98
 msgid "You cannot schedule evaluations for past slots."
 msgstr ""
 
@@ -4122,7 +4088,7 @@ msgstr ""
 msgid "You don't have any notifications."
 msgstr ""
 
-#: templates/quiz/quiz.js:136
+#: templates/quiz/quiz.js:137
 msgid "You got"
 msgstr ""
 
@@ -4176,11 +4142,17 @@ msgstr ""
 
 #: lms/notification/certificate_request_creation/certificate_request_creation.html:7
 #: lms/notification/certificate_request_reminder/certificate_request_reminder.html:6
+#: templates/emails/certificate_request_notification.html:2
 msgid "Your evaluation for the course {0} has been scheduled on {1} at {2} {3}."
 msgstr ""
 
+#: lms/doctype/lms_certificate_request/lms_certificate_request.py:119
+msgid "Your evaluation slot has been booked"
+msgstr ""
+
 #: lms/notification/certificate_request_creation/certificate_request_creation.html:8
 #: lms/notification/certificate_request_reminder/certificate_request_reminder.html:7
+#: templates/emails/certificate_request_notification.html:3
 msgid "Your evaluator is {0}"
 msgstr ""
 
@@ -4188,7 +4160,7 @@ msgstr ""
 msgid "Your request to join us as a mentor for the course"
 msgstr ""
 
-#: templates/quiz/quiz.js:136
+#: templates/quiz/quiz.js:140
 msgid "Your score is"
 msgstr ""
 
@@ -4201,7 +4173,7 @@ msgstr ""
 msgid "cancel your application"
 msgstr ""
 
-#: templates/quiz/quiz.js:136
+#: templates/quiz/quiz.js:137
 msgid "correct answers"
 msgstr ""
 
@@ -4217,7 +4189,7 @@ msgstr ""
 msgid "of"
 msgstr ""
 
-#: templates/quiz/quiz.js:136
+#: templates/quiz/quiz.js:141
 msgid "out of"
 msgstr ""
 
@@ -4261,7 +4233,7 @@ msgstr ""
 msgid "{0} is already certified for the course {1}"
 msgstr ""
 
-#: lms/utils.py:696
+#: lms/utils.py:689
 msgid "{0} mentioned you in a comment"
 msgstr ""
 
@@ -4269,7 +4241,7 @@ msgstr ""
 msgid "{0} mentioned you in a comment in your batch."
 msgstr ""
 
-#: lms/utils.py:649 lms/utils.py:655
+#: lms/utils.py:642 lms/utils.py:648
 msgid "{0} mentioned you in a comment in {1}"
 msgstr ""
 

From cdd46667f332ea9e23922c29e5259437d54219a0 Mon Sep 17 00:00:00 2001
From: Jannat Patel <pateljannat2308@gmail.com>
Date: Mon, 19 Aug 2024 11:47:17 +0530
Subject: [PATCH 06/10] feat: add new member

---
 frontend/src/components/Members.vue         | 248 ++++++++++++--------
 frontend/src/components/Modals/Settings.vue |  56 +++--
 frontend/src/components/SettingDetails.vue  |  19 +-
 lms/lms/api.py                              |  20 +-
 lms/overrides/user.py                       |   3 +
 5 files changed, 212 insertions(+), 134 deletions(-)

diff --git a/frontend/src/components/Members.vue b/frontend/src/components/Members.vue
index e36c7726..ee105055 100644
--- a/frontend/src/components/Members.vue
+++ b/frontend/src/components/Members.vue
@@ -1,55 +1,83 @@
 <template>
-    <div class="text-base p-4">
-        <div class="flex items-center justify-between">
-            <div>
-                <div class="font-semibold mb-1">
-                    {{ __(label) }}
-                </div>
-                <div class="text-xs text-gray-600">
-                    {{ __(description) }}
-                </div>
-            </div>
-            <FormControl v-model="search" :placeholder="__('Search')" type="text" :debounce="300"/>
-        </div>
-        <div class="my-4">
+	<div class="text-base p-4">
+		<div class="flex items-center justify-between">
+			<div>
+				<div class="font-semibold mb-1">
+					{{ __(label) }}
+				</div>
+				<div class="text-xs text-gray-600">
+					{{ __(description) }}
+				</div>
+			</div>
+			<div class="flex item-center space-x-2">
+				<FormControl
+					v-model="search"
+					:placeholder="__('Search')"
+					type="text"
+					:debounce="300"
+				/>
+				<Button @click="() => (showForm = true)">
+					<template #icon>
+						<component :is="icons['Plus']" class="h-3 w-3 stroke-1.5" />
+					</template>
+				</Button>
+			</div>
+		</div>
+		<div class="my-4">
+			<!-- Form to add new member -->
+			<div v-if="showForm" class="flex items-center space-x-2 mb-4">
+				<FormControl
+					v-model="member.email"
+					:placeholder="__('Email')"
+					type="email"
+					class="w-full"
+				/>
+				<FormControl
+					v-model="member.first_name"
+					:placeholder="__('First Name')"
+					type="test"
+					class="w-full"
+				/>
+				<Button @click="addMember()" variant="subtle">
+					{{ __('Add') }}
+				</Button>
+			</div>
 
-            <!-- Form to add new member -->
-            <div v-if="showForm" class="flex items-center space-x-2 mb-4">
-                <FormControl v-model="member.email" :placeholder="__('Email')" type="email" class="w-full"/>
-                <FormControl v-model="member.first_name" :placeholder="__('First Name')" type="test" class="w-full"/>
-                <Button @click="addMember" variant="subtle">
-                    {{ __('Add') }}
-                </Button>
-            </div>
-
-            <!-- Member list -->
-            <div v-for="member in memberList" class="grid grid-cols-5 grid-flow-row py-2 cursor-pointer">
-                <div @click="openProfile(member.username)" class="flex items-center space-x-2 col-span-2">
-                    <Avatar :image="member.user_image" :label="member.full_name" size="sm" />
-                    <div>
-                       {{ member.full_name }}
-                    </div>
-                </div>
-                <div class="text-sm text-gray-700 col-span-2">
-                    {{ member.name }}
-                </div>
-                <div class="text-sm text-gray-700 justify-self-end">
-                    {{ getRole(member.role) }}
-                </div>
-            </div>
-        </div>
-        <div v-if="hasNextPage" class="flex justify-center">
-                <Button variant="solid" @click="members.reload()">
-                    <template #prefix>
-                        <component
-                            :is="icons['RefreshCw']"
-                            class="h-3 w-3 stroke-1.5"
-                        />
-                    </template>
-                    {{ __('Load More') }}
-                </Button>
-            </div>
-    </div>
+			<!-- Member list -->
+			<div
+				v-for="member in memberList"
+				class="grid grid-cols-5 grid-flow-row py-2 cursor-pointer"
+			>
+				<div
+					@click="openProfile(member.username)"
+					class="flex items-center space-x-2 col-span-2"
+				>
+					<Avatar
+						:image="member.user_image"
+						:label="member.full_name"
+						size="sm"
+					/>
+					<div>
+						{{ member.full_name }}
+					</div>
+				</div>
+				<div class="text-sm text-gray-700 col-span-2">
+					{{ member.name }}
+				</div>
+				<div class="text-sm text-gray-700 justify-self-end">
+					{{ getRole(member.role) }}
+				</div>
+			</div>
+		</div>
+		<div v-if="hasNextPage" class="flex justify-center">
+			<Button variant="solid" @click="members.reload()">
+				<template #prefix>
+					<component :is="icons['RefreshCw']" class="h-3 w-3 stroke-1.5" />
+				</template>
+				{{ __('Load More') }}
+			</Button>
+		</div>
+	</div>
 </template>
 <script setup lang="ts">
 import { createResource, Avatar, Button, FormControl } from 'frappe-ui'
@@ -58,72 +86,98 @@ import { useRouter } from 'vue-router'
 import { ref, watch, reactive } from 'vue'
 
 const router = useRouter()
-const show = defineModel("show")
+const show = defineModel('show')
 const search = ref('')
 const start = ref(0)
 const memberList = ref([])
 const hasNextPage = ref(false)
-const showForm = ref(true)
+const showForm = ref(false)
+
 const member = reactive({
-    email: '',
-    first_name: '',
+	email: '',
+	first_name: '',
 })
 
 const props = defineProps({
-    label: {
-        type: String,
-        required: true,
-    },
-    description: {
-        type: String,
-        default: '',
-    },
-    show: {
-        type: Boolean,
-    },
+	label: {
+		type: String,
+		required: true,
+	},
+	description: {
+		type: String,
+		default: '',
+	},
+	show: {
+		type: Boolean,
+	},
 })
 
 const members = createResource({
-    url: "lms.lms.api.get_members",
-    makeParams: () => {
-        return {
-            search: search.value,
-            start: start.value
-        }
-    },
-    onSuccess(data) {
-        memberList.value = memberList.value.concat(data)
-        start.value = start.value + 20
-        hasNextPage.value = data.length === 20
-    },
-    auto: true,
+	url: 'lms.lms.api.get_members',
+	makeParams: () => {
+		return {
+			search: search.value,
+			start: start.value,
+		}
+	},
+	onSuccess(data) {
+		memberList.value = memberList.value.concat(data)
+		start.value = start.value + 20
+		hasNextPage.value = data.length === 20
+	},
+	auto: true,
 })
 
 const openProfile = (username) => {
-    show.value = false
-    router.push({
-        name: 'Profile',
-        params: {
-            username: username,
-        },
-    })
+	show.value = false
+	router.push({
+		name: 'Profile',
+		params: {
+			username: username,
+		},
+	})
 }
 
+const newMember = createResource({
+	url: 'frappe.client.insert',
+	makeParams(values) {
+		return {
+			doc: {
+				doctype: 'User',
+				first_name: member.first_name,
+				email: member.email,
+			},
+		}
+	},
+	auto: false,
+	onSuccess(data) {
+		show.value = false
+		router.push({
+			name: 'Profile',
+			params: {
+				username: data.username,
+			},
+		})
+	},
+})
+
+const addMember = () => {
+	newMember.reload()
+}
 
 watch(search, () => {
-    memberList.value = []
-    start.value = 0
-    members.reload()
+	memberList.value = []
+	start.value = 0
+	members.reload()
 })
 
 const getRole = (role) => {
-   const map = {
-       'LMS Student': 'Student',
-       'Course Creator': 'Instructor',
-       'Moderator': 'Moderator',
-       'Batch Evaluator': 'Evaluator'
-   }
-   return map[role]
+	const map = {
+		'LMS Student': 'Student',
+		'Course Creator': 'Instructor',
+		Moderator: 'Moderator',
+		'Batch Evaluator': 'Evaluator',
+	}
+	return map[role]
 }
-
-</script>
\ No newline at end of file
+</script>
diff --git a/frontend/src/components/Modals/Settings.vue b/frontend/src/components/Modals/Settings.vue
index bbb166bb..91b2a381 100644
--- a/frontend/src/components/Modals/Settings.vue
+++ b/frontend/src/components/Modals/Settings.vue
@@ -28,13 +28,22 @@
 						</nav>
 					</div>
 				</div>
-				<div v-if="activeTab && data.doc" class="flex flex-1 flex-col overflow-y-auto">
-					<Members v-if="activeTab.label === 'Members'" :label="activeTab.label" :description="activeTab.description" v-model:show="show"/>
+				<div
+					v-if="activeTab && data.doc"
+					class="flex flex-1 flex-col overflow-y-auto"
+				>
+					<Members
+						v-if="activeTab.label === 'Members'"
+						:label="activeTab.label"
+						:description="activeTab.description"
+						v-model:show="show"
+					/>
 					<SettingDetails
 						v-else
 						:fields="activeTab.fields"
-						:data="data"
 						:label="activeTab.label"
+						:description="activeTab.description"
+						:data="data"
 					/>
 				</div>
 			</div>
@@ -66,9 +75,16 @@ const tabs = computed(() => {
 			label: 'Settings',
 			hideLabel: true,
 			items: [
+				{
+					label: 'Members',
+					description: 'Manage the members of your learning system',
+					icon: 'UserRoundPlus',
+				},
 				{
 					label: 'Payment Gateway',
 					icon: 'DollarSign',
+					description:
+						'Configure the payment gateway and other payment related settings',
 					fields: [
 						{
 							label: 'Razorpay Key',
@@ -115,6 +131,7 @@ const tabs = computed(() => {
 				{
 					label: 'Sidebar',
 					icon: 'PanelLeftIcon',
+					description: 'Customize the sidebar as per your needs',
 					fields: [
 						{
 							label: 'Courses',
@@ -160,6 +177,7 @@ const tabs = computed(() => {
 				{
 					label: 'Email Templates',
 					icon: 'MailPlus',
+					description: 'Create email templates with the content you want',
 					fields: [
 						{
 							label: 'Batch Confirmation Template',
@@ -190,9 +208,11 @@ const tabs = computed(() => {
 				{
 					label: 'Signup',
 					icon: 'LogIn',
+					description:
+						'Customize the signup page to inform users about your terms and policies',
 					fields: [
 						{
-							label: 'Show terms of use on signup page',
+							label: 'Show terms of use on signup',
 							name: 'terms_of_use',
 							type: 'checkbox',
 						},
@@ -203,15 +223,7 @@ const tabs = computed(() => {
 							doctype: 'Web Page',
 						},
 						{
-							label: 'Ask user category during signup',
-							name: 'user_category',
-							type: 'checkbox',
-						},
-						{
-							type: 'Column Break',
-						},
-						{
-							label: 'Show privacy policy on signup page',
+							label: 'Show privacy policy on signup',
 							name: 'privacy_policy',
 							type: 'checkbox',
 						},
@@ -225,7 +237,7 @@ const tabs = computed(() => {
 							type: 'Column Break',
 						},
 						{
-							label: 'Show cookie policy on signup page',
+							label: 'Show cookie policy on signup',
 							name: 'cookie_policy',
 							type: 'checkbox',
 						},
@@ -235,21 +247,15 @@ const tabs = computed(() => {
 							type: 'Link',
 							doctype: 'Web Page',
 						},
+						{
+							label: 'Ask user category during signup',
+							name: 'user_category',
+							type: 'checkbox',
+						},
 					],
 				},
 			],
 		},
-		{
-			label: 'Settings',
-			hideLabel: true,
-			items: [
-			{
-				label: 'Members',
-				description: 'Manage the members of your learning system',
-				icon: "UserRoundPlus",
-			},
-			],
-		},
 	]
 
 	return _tabs.map((tab) => {
diff --git a/frontend/src/components/SettingDetails.vue b/frontend/src/components/SettingDetails.vue
index 2fc83667..0fff5a58 100644
--- a/frontend/src/components/SettingDetails.vue
+++ b/frontend/src/components/SettingDetails.vue
@@ -1,6 +1,14 @@
 <template>
-	<div class="flex flex-col justify-between h-full p-8">
-		<div class="flex space-x-8">
+	<div class="flex flex-col justify-between h-full p-4">
+		<div>
+			<div class="font-semibold mb-1">
+				{{ __(label) }}
+			</div>
+			<div class="text-xs text-gray-600">
+				{{ __(description) }}
+			</div>
+		</div>
+		<div class="flex space-x-8 my-5">
 			<div v-for="(column, index) in columns" :key="index">
 				<div class="flex flex-col space-y-4 w-60">
 					<div v-for="field in column">
@@ -43,6 +51,13 @@ const props = defineProps({
 		type: Object,
 		required: true,
 	},
+	label: {
+		type: String,
+		required: true,
+	},
+	description: {
+		type: String,
+	},
 })
 
 const columns = computed(() => {
diff --git a/lms/lms/api.py b/lms/lms/api.py
index 583125bc..c26e1739 100644
--- a/lms/lms/api.py
+++ b/lms/lms/api.py
@@ -563,19 +563,22 @@ def get_categories(doctype, filters):
 
 	return categoryOptions
 
+
 @frappe.whitelist()
 def get_members(start=0, search=""):
-	filters = {
-			"enabled": 1,
-			"name": ["not in", ["Administrator", "Guest"]]
-		}
-	
+	filters = {"enabled": 1, "name": ["not in", ["Administrator", "Guest"]]}
+
 	if search:
 		filters["full_name"] = ["like", f"%{search}%"]
 
 	print(filters)
-	members = frappe.get_all("User", filters=filters, fields=["name", "full_name", "user_image", "username"],
-	page_length=20, start=start)
+	members = frappe.get_all(
+		"User",
+		filters=filters,
+		fields=["name", "full_name", "user_image", "username"],
+		page_length=20,
+		start=start,
+	)
 
 	for member in members:
 		roles = frappe.get_roles(member.name)
@@ -589,6 +592,3 @@ def get_members(start=0, search=""):
 			member.role = "LMS Student"
 
 	return members
-		
-
-	
\ No newline at end of file
diff --git a/lms/overrides/user.py b/lms/overrides/user.py
index b7659493..da1d416b 100644
--- a/lms/overrides/user.py
+++ b/lms/overrides/user.py
@@ -16,6 +16,9 @@ class CustomUser(User):
 		super().validate()
 		self.validate_username_duplicates()
 
+	def after_insert(self):
+		self.add_roles("LMS Student")
+
 	def validate_username_duplicates(self):
 		while not self.username or self.username_exists():
 			self.username = append_number_if_name_exists(

From a3b8ed8f918539d226850446702a156143abc541 Mon Sep 17 00:00:00 2001
From: Jannat Patel <pateljannat2308@gmail.com>
Date: Mon, 19 Aug 2024 12:03:32 +0530
Subject: [PATCH 07/10] fix: documented the api

---
 frontend/src/components/Members.vue         | 6 +++---
 frontend/src/components/Modals/Settings.vue | 2 +-
 lms/lms/api.py                              | 7 ++++++-
 package.json                                | 2 +-
 4 files changed, 11 insertions(+), 6 deletions(-)

diff --git a/frontend/src/components/Members.vue b/frontend/src/components/Members.vue
index ee105055..a6cc8d9b 100644
--- a/frontend/src/components/Members.vue
+++ b/frontend/src/components/Members.vue
@@ -18,7 +18,7 @@
 				/>
 				<Button @click="() => (showForm = true)">
 					<template #icon>
-						<component :is="icons['Plus']" class="h-3 w-3 stroke-1.5" />
+                        <Plus class="h-3 w-3 stroke-1.5" />
 					</template>
 				</Button>
 			</div>
@@ -72,7 +72,7 @@
 		<div v-if="hasNextPage" class="flex justify-center">
 			<Button variant="solid" @click="members.reload()">
 				<template #prefix>
-					<component :is="icons['RefreshCw']" class="h-3 w-3 stroke-1.5" />
+                    <RefreshCw class="h-3 w-3 stroke-1.5" />
 				</template>
 				{{ __('Load More') }}
 			</Button>
@@ -81,9 +81,9 @@
 </template>
 <script setup lang="ts">
 import { createResource, Avatar, Button, FormControl } from 'frappe-ui'
-import * as icons from 'lucide-vue-next'
 import { useRouter } from 'vue-router'
 import { ref, watch, reactive } from 'vue'
+import { RefreshCw, Plus } from "lucide-vue-next"
 
 const router = useRouter()
 const show = defineModel('show')
diff --git a/frontend/src/components/Modals/Settings.vue b/frontend/src/components/Modals/Settings.vue
index 91b2a381..ec56edbe 100644
--- a/frontend/src/components/Modals/Settings.vue
+++ b/frontend/src/components/Modals/Settings.vue
@@ -1,6 +1,6 @@
 <template>
 	<Dialog v-model="show" :options="{ size: '3xl' }">
-		<template #body="{ close }">
+		<template #body>
 			<div class="flex h-[calc(100vh_-_8rem)]">
 				<div class="flex w-52 shrink-0 flex-col bg-gray-50 p-2">
 					<h1 class="mb-3 px-2 pt-2 text-lg font-semibold">
diff --git a/lms/lms/api.py b/lms/lms/api.py
index c26e1739..8b235558 100644
--- a/lms/lms/api.py
+++ b/lms/lms/api.py
@@ -566,12 +566,17 @@ def get_categories(doctype, filters):
 
 @frappe.whitelist()
 def get_members(start=0, search=""):
+	""" Get members for the given search term and start index.
+		Args: start (int): Start index for the query.
+			search (str): Search term to filter the results.
+		Returns: List of members.
+	"""
+	
 	filters = {"enabled": 1, "name": ["not in", ["Administrator", "Guest"]]}
 
 	if search:
 		filters["full_name"] = ["like", f"%{search}%"]
 
-	print(filters)
 	members = frappe.get_all(
 		"User",
 		filters=filters,
diff --git a/package.json b/package.json
index 923d2f3e..f5231fac 100644
--- a/package.json
+++ b/package.json
@@ -2,7 +2,7 @@
   "name": "frappe_lms",
   "version": "1.0.0",
   "description": "Easy to use, open-source, Learning Management System",
-  "workspaces": [
+  "workspaces1": [
     "frappe-ui",
     "frontend"
   ],

From 63f857b8fcc49f2622761a7ac69da2821dd029b0 Mon Sep 17 00:00:00 2001
From: Jannat Patel <pateljannat2308@gmail.com>
Date: Mon, 19 Aug 2024 12:12:51 +0530
Subject: [PATCH 08/10] fix: linters

---
 frontend/src/components/Members.vue |  6 +++---
 lms/lms/api.py                      | 10 +++++-----
 2 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/frontend/src/components/Members.vue b/frontend/src/components/Members.vue
index a6cc8d9b..22dc2074 100644
--- a/frontend/src/components/Members.vue
+++ b/frontend/src/components/Members.vue
@@ -18,7 +18,7 @@
 				/>
 				<Button @click="() => (showForm = true)">
 					<template #icon>
-                        <Plus class="h-3 w-3 stroke-1.5" />
+						<Plus class="h-3 w-3 stroke-1.5" />
 					</template>
 				</Button>
 			</div>
@@ -72,7 +72,7 @@
 		<div v-if="hasNextPage" class="flex justify-center">
 			<Button variant="solid" @click="members.reload()">
 				<template #prefix>
-                    <RefreshCw class="h-3 w-3 stroke-1.5" />
+					<RefreshCw class="h-3 w-3 stroke-1.5" />
 				</template>
 				{{ __('Load More') }}
 			</Button>
@@ -83,7 +83,7 @@
 import { createResource, Avatar, Button, FormControl } from 'frappe-ui'
 import { useRouter } from 'vue-router'
 import { ref, watch, reactive } from 'vue'
-import { RefreshCw, Plus } from "lucide-vue-next"
+import { RefreshCw, Plus } from 'lucide-vue-next'
 
 const router = useRouter()
 const show = defineModel('show')
diff --git a/lms/lms/api.py b/lms/lms/api.py
index 8b235558..df2272cd 100644
--- a/lms/lms/api.py
+++ b/lms/lms/api.py
@@ -566,12 +566,12 @@ def get_categories(doctype, filters):
 
 @frappe.whitelist()
 def get_members(start=0, search=""):
-	""" Get members for the given search term and start index.
-		Args: start (int): Start index for the query.
-			search (str): Search term to filter the results.
-		Returns: List of members.
+	"""Get members for the given search term and start index.
+	Args: start (int): Start index for the query.
+	    search (str): Search term to filter the results.
+	Returns: List of members.
 	"""
-	
+
 	filters = {"enabled": 1, "name": ["not in", ["Administrator", "Guest"]]}
 
 	if search:

From 0be5439e8157cbd30ca4a28a5823aa1091c51a5e Mon Sep 17 00:00:00 2001
From: Jannat Patel <pateljannat2308@gmail.com>
Date: Mon, 19 Aug 2024 12:31:46 +0530
Subject: [PATCH 09/10] fix: tests

---
 .../invite_request/test_invite_request.py     | 72 +------------------
 1 file changed, 1 insertion(+), 71 deletions(-)

diff --git a/lms/lms/doctype/invite_request/test_invite_request.py b/lms/lms/doctype/invite_request/test_invite_request.py
index b9cac5d3..0cc82050 100644
--- a/lms/lms/doctype/invite_request/test_invite_request.py
+++ b/lms/lms/doctype/invite_request/test_invite_request.py
@@ -11,74 +11,4 @@ from lms.lms.doctype.invite_request.invite_request import (
 
 
 class TestInviteRequest(unittest.TestCase):
-	@classmethod
-	def setUpClass(self):
-		create_invite_request("test_invite@example.com")
-
-	def test_create_invite_request(self):
-		if frappe.db.exists("Invite Request", {"invite_email": "test_invite@example.com"}):
-			invite = frappe.db.get_value(
-				"Invite Request",
-				filters={"invite_email": "test_invite@example.com"},
-				fieldname=["invite_email", "status", "signup_email"],
-				as_dict=True,
-			)
-			self.assertEqual(invite.status, "Approved")
-			self.assertEqual(invite.signup_email, None)
-
-	def test_create_invite_request_update(self):
-		if frappe.db.exists("Invite Request", {"invite_email": "test_invite@example.com"}):
-
-			data = {
-				"signup_email": "test_invite@example.com",
-				"username": "test_invite",
-				"full_name": "Test Invite",
-				"password": "Test@invite",
-				"invite_code": frappe.db.get_value(
-					"Invite Request", {"invite_email": "test_invite@example.com"}, "name"
-				),
-			}
-
-			update_invite(data)
-			invite = frappe.db.get_value(
-				"Invite Request",
-				filters={"invite_email": "test_invite@example.com"},
-				fieldname=[
-					"invite_email",
-					"status",
-					"signup_email",
-					"full_name",
-					"username",
-					"invite_code",
-					"name",
-				],
-				as_dict=True,
-			)
-			self.assertEqual(invite.signup_email, "test_invite@example.com")
-			self.assertEqual(invite.full_name, "Test Invite")
-			self.assertEqual(invite.username, "test_invite")
-			self.assertEqual(invite.invite_code, invite.name)
-			self.assertEqual(invite.status, "Registered")
-
-			user = frappe.db.get_value(
-				"User",
-				"test_invite@example.com",
-				fieldname=["first_name", "username", "send_welcome_email", "user_type"],
-				as_dict=True,
-			)
-			self.assertTrue(user)
-			self.assertEqual(user.first_name, invite.full_name.split(" ")[0])
-			self.assertEqual(user.username, invite.username)
-			self.assertEqual(user.send_welcome_email, 0)
-			self.assertEqual(user.user_type, "Website User")
-
-	@classmethod
-	def tearDownClass(self):
-		if frappe.db.exists("User", "test_invite@example.com"):
-			frappe.delete_doc("User", "test_invite@example.com")
-
-		invite_request = frappe.db.exists(
-			"Invite Request", {"invite_email": "test_invite@example.com"}
-		)
-		if invite_request:
-			frappe.delete_doc("Invite Request", invite_request)
+	pass

From 98b8464e1a633c4fb9ed4fdd35db327da6d49745 Mon Sep 17 00:00:00 2001
From: Jannat Patel <pateljannat2308@gmail.com>
Date: Mon, 19 Aug 2024 13:07:48 +0530
Subject: [PATCH 10/10] fix: ui test

---
 .github/workflows/ui-tests.yml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/.github/workflows/ui-tests.yml b/.github/workflows/ui-tests.yml
index b9aa4614..f093e55a 100644
--- a/.github/workflows/ui-tests.yml
+++ b/.github/workflows/ui-tests.yml
@@ -99,6 +99,7 @@ jobs:
           cd ~/frappe-bench/
           bench --site lms.test execute frappe.utils.install.complete_setup_wizard
           bench --site lms.test execute frappe.tests.ui_test_helpers.create_test_user
+          bench --site lms.test set-password frappe@example.com admin
 
       - name: cypress pre-requisites
         run: |