MIDNIGHT/frappe-lms
2
0
Fork 0
mirror of https://github.com/frappe/lms.git synced 2026-04-23 00:38:00 +03:00
Code Issues Packages Projects Releases Wiki Activity

fix: dont capture progress of demo course for analytics

Browse Source 
(cherry picked from commit a6da65ab99)

# Conflicts:
#	lms/lms/utils.py
This commit is contained in:
Jannat Patel
2026-03-09 16:22:21 +05:30
committed by Mergify
parent dd96f18d2d
commit 3801816535
3 changed files with 30 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
lms/demo/demo_data.py
View File

@@ -172,7 +172,7 @@ def create_intro_lesson_1(course):
title = "What is a Learning Management System?"
chapter = get_chapter(course, "Introduction")
content = """
{"time":1772449622100,"blocks":[{"id":"vYTdcXYVgI","type":"embed","data":{"service":"youtube","source":"http://youtube.com/watch?v=SauviPVDItU","embed":"SauviPVDItU","caption":""}}],"version":"2.29.0"}
{"time":1772449622100,"blocks":[{"id":"vYTdcXYVgI","type":"embed","data":{"service":"youtube","source":"https://www.youtube.com/watch?v=-Ulzqjj49lk","embed":"-Ulzqjj49lk","caption":""}}],"version":"2.29.0"}
"""
create_lesson(course, chapter, title, content)
@@ -181,7 +181,7 @@ def create_intro_lesson_2(course):
title = "What is Frappe Learning?"
chapter = get_chapter(course, "Introduction")
content = """
{"time":1772449622100,"blocks":[{"id":"vYTdcXYVgI","type":"embed","data":{"service":"youtube","source":"http://youtube.com/watch?v=SauviPVDItU","embed":"SauviPVDItU","caption":""}}],"version":"2.29.0"}
{"time":1772449622100,"blocks":[{"id":"vYTdcXYVgI","type":"embed","data":{"service":"youtube","source":"https://www.youtube.com/watch?v=VIt_bsbBjLI","embed":"VIt_bsbBjLI","caption":""}}],"version":"2.29.0"}
"""
create_lesson(course, chapter, title, content)

9
lms/lms/doctype/course_lesson/course_lesson.py
View File

@@ -9,7 +9,7 @@ from frappe.model.document import Document
from frappe.realtime import get_website_room
from frappe.utils.telemetry import capture
from lms.lms.utils import get_course_progress, recalculate_course_progress
from lms.lms.utils import get_course_progress, is_demo_course, recalculate_course_progress
from ...md import find_macros
@@ -127,7 +127,8 @@ def save_progress(lesson: str, course: str, scorm_details: dict = None):
)
progress = get_course_progress(course)
capture_progress_for_analytics()
if not is_demo_course(course):
capture("course_progress", "lms")
# Had to get doc, as on_change doesn't trigger when you use set_value. The trigger is necessary for badge to get assigned.
enrollment = frappe.get_doc("LMS Enrollment", membership)
@@ -145,10 +146,6 @@ def save_progress(lesson: str, course: str, scorm_details: dict = None):
return progress
def capture_progress_for_analytics():
capture("course_progress", "lms")
def get_quiz_progress(lesson):
lesson_details = frappe.db.get_value("Course Lesson", lesson, ["body", "content"], as_dict=1)
quizzes = []

25
lms/lms/utils.py
View File

@@ -2328,3 +2328,28 @@ def recalculate_course_progress(course: str, member: str):
)
frappe.db.set_value("LMS Enrollment", membership, "progress", progress)
update_program_progress(member)
<<<<<<< HEAD
=======
def get_field_meta(doctype, fieldnames):
"""Returns field metadata for 'fieldnames' from 'doctype'"""
meta = frappe.get_meta(doctype)
fieldnames_meta = {}
for fieldname in fieldnames:
field = meta.get_field(fieldname)
if field:
fieldnames_meta[fieldname] = {
"reqd": field.reqd,
"default": field.default,
"description": field.description,
}
return fieldnames_meta
def is_demo_course(course: str) -> bool:
title = frappe.db.get_value("LMS Course", course, "title")
return title == "A guide to Frappe Learning"
>>>>>>> a6da65ab (fix: dont capture progress of demo course for analytics)