fix: show first chapter and question form by default

lms/www/batch/learn.html

@@ -33,7 +33,9 @@
             </div>
             <div class="lesson-pagination-parent">
                 {{ LessonContent(lesson) }}
-                {% if not lesson.edit_mode %} {{ Discussions() }} {% endif %}
+                {% if not lesson.edit_mode and course.staus == "Approved" and not course.upcoming %}
+                {{ Discussions() }}
+                {% endif %}
             </div>
         </div>
     </div>

lms/www/batch/learn.js

@@ -498,8 +498,14 @@ const save_lesson = (e) => {
             "idx": $("#title").data("index"),
             "lesson": lesson ? lesson : ""
         },
-        callback: (data) => {
-            window.location.href = window.location.href.split("?")[0];
+        callback: (data) => {;
+            frappe.show_alert({
+                message: __("Saved"),
+                indicator: "green",
+            });
+            setTimeout(() => {
+                window.location.href = window.location.href.split("?")[0];
+            }, 1000);
         }
     });
 };

lms/www/batch/learn.py

@@ -24,7 +24,7 @@ def get_context(context):
 
     context.lesson = get_current_lesson_details(lesson_number, context)
     instructor = is_instructor(context.course.name)
-    context.show_lesson = context.membership or context.lesson.include_in_preview or instructor or has_course_moderator_role()
+    context.show_lesson = context.membership or (context.lesson and context.lesson.include_in_preview) or instructor or has_course_moderator_role()
 
     if not context.lesson:
         context.lesson = frappe._dict()

lms/www/batch/quiz.html

@@ -40,7 +40,7 @@
     {% for question in quiz.questions %}
     <div class="quiz-card">
         <div contenteditable="true" data-placeholder="{{ _('Question') }}" data-question="{{ question.name }}"
-        class="question mb-4">{% if question.question %} {{ question.question }} {% endif %}</div>
+        class="question mb-4 req">{% if question.question %} {{ question.question }} {% endif %}</div>
 
         {% for i in range(1,5) %}
         {% set num = frappe.utils.cstr(i) %}

lms/www/batch/quiz.js

@@ -1,7 +1,11 @@
 frappe.ready(() => {
 
+    if(!$(".quiz-card").length) {
+        add_question();
+    }
+
     $(".btn-question").click((e) => {
-        add_question(e);
+        add_question();
     });
 
     $(".btn-save-question").click((e) => {
@@ -17,10 +21,15 @@ frappe.ready(() => {
 });
 
 
-const add_question = (e) => {
+const add_question = () => {
+    if ($(".new-quiz-card").length) {
+        scroll_to_question_container();
+        return;
+    }
+
     let add_after = $(".quiz-card").length ? $(".quiz-card:last") : $("#quiz-title");
-    let question_template = `<div class="quiz-card">
-            <div contenteditable="true" data-placeholder="${__("Question")}" class="question mb-4"></div>
+    let question_template = `<div class="quiz-card new-quiz-card">
+            <div contenteditable="true" data-placeholder="${__("Question")}" class="question req mb-4"></div>
         </div>`;
     $(question_template).insertAfter(add_after);
     get_question_template();
@@ -109,3 +118,11 @@ const get_questions = () => {
 
     return questions;
 };
+
+
+const scroll_to_question_container = () => {
+    $([document.documentElement, document.body]).animate({
+        scrollTop: $(".new-quiz-card").offset().top
+    }, 1000);
+    $(".new-quiz-card").find(".question").focus();
+}

lms/www/courses/course.html

@@ -22,7 +22,7 @@
                     {{ Description(course) }}
                     {{ Save(course) }}
                     {{ widgets.CourseOutline(course=course, membership=membership, is_user_interested=is_user_interested) }}
-                    {% if not course.edit_mode %}
+                    {% if not course.edit_mode and course.staus == "Approved" and not course.upcoming %}
                     {{ widgets.Reviews(course=course, membership=membership) }}
                     {% endif %}
                 </div>