diff --git a/lms/lms/doctype/lms_course/lms_course.py b/lms/lms/doctype/lms_course/lms_course.py index 0f497eb3..f6008f3d 100644 --- a/lms/lms/doctype/lms_course/lms_course.py +++ b/lms/lms/doctype/lms_course/lms_course.py @@ -194,7 +194,7 @@ def submit_for_review(course): @frappe.whitelist() -def save_course(tags, title, short_introduction, video_link, description, course, image=None): +def save_course(tags, title, short_introduction, video_link, description, course, published, upcoming, image=None): if course: doc = frappe.get_doc("LMS Course", course) else: @@ -208,7 +208,9 @@ def save_course(tags, title, short_introduction, video_link, description, course "video_link": video_link, "image": image, "description": description, - "tags": tags + "tags": tags, + "published": published, + "upcoming": upcoming }) doc.save(ignore_permissions=True) return doc.name diff --git a/lms/lms/widgets/CourseOutline.html b/lms/lms/widgets/CourseOutline.html index d2f47904..2aec5324 100644 --- a/lms/lms/widgets/CourseOutline.html +++ b/lms/lms/widgets/CourseOutline.html @@ -55,7 +55,7 @@ {% set active = membership.current_lesson == lesson.name %}
- {% if membership or lesson.include_in_preview or is_instructor %} + {% if membership or lesson.include_in_preview or is_instructor or has_course_moderator_role() %} { }; -dahanukar -sandesh -chetna + diff --git a/lms/public/css/style.css b/lms/public/css/style.css index 2fe71a33..b8b2b9f7 100644 --- a/lms/public/css/style.css +++ b/lms/public/css/style.css @@ -74,7 +74,6 @@ input[type=checkbox] { .common-page-style { padding: 2rem 0 5rem; - min-height: 60vh; padding-top: 3rem; background-color: var(--bg-color); } diff --git a/lms/www/batch/learn.html b/lms/www/batch/learn.html index 78896567..629e4ca0 100644 --- a/lms/www/batch/learn.html +++ b/lms/www/batch/learn.html @@ -57,6 +57,7 @@ {% macro LessonContent(lesson) %} {% set instructors = get_instructors(course.name) %} {% set is_instructor = is_instructor(course.name) %} +
@@ -70,7 +71,7 @@ {{ _("COMPLETED") }} - {% if is_instructor and not lesson.edit_mode %} + {% if (is_instructor or has_course_moderator_role()) and not lesson.edit_mode %} {% endif %}
@@ -105,7 +106,7 @@
- {% if membership or lesson.include_in_preview or is_instructor %} + {% if show_lesson %} {% if is_instructor and not lesson.include_in_preview and not lesson.edit_mode %}
@@ -190,9 +191,9 @@
{% if lesson.quiz_id %}{{ lesson.quiz_id }}{% endif %}
-
diff --git a/lms/www/courses/course.html b/lms/www/courses/course.html index 75d26a28..7472880a 100644 --- a/lms/www/courses/course.html +++ b/lms/www/courses/course.html @@ -18,6 +18,7 @@
{{ CourseHeaderOverlay(course) }} + {{ CourseSettings(course) }} {{ Description(course) }} {{ Save(course) }} {{ widgets.CourseOutline(course=course, membership=membership, is_user_interested=is_user_interested) }} @@ -210,6 +211,25 @@ {% endmacro %} + +{% macro CourseSettings(course) %} + + {% if has_course_moderator_role() %} +
+ + +
+ {% endif %} + +{% endmacro %} + + {% macro Save(course) %} {% if course.edit_mode %} diff --git a/lms/www/courses/course.js b/lms/www/courses/course.js index 083391f7..0669f119 100644 --- a/lms/www/courses/course.js +++ b/lms/www/courses/course.js @@ -339,7 +339,9 @@ const save_course = (e) => { "video_link": $("#video-link").text(), "image": $("#image").attr("href"), "description": $("#description").text(), - "course": $("#title").data("course") ? $("#title").data("course") : "" + "course": $("#title").data("course") ? $("#title").data("course") : "", + "published": $("#published").prop("checked") ? 1 : 0, + "upcoming": $("#upcoming").prop("checked") ? 1 : 0 }, callback: (data) => { frappe.show_alert({