Merge pull request #404 from pateljannat/start-learning-issue

lms/lms/widgets/CourseOutline.html

@@ -111,7 +111,7 @@
 
 </div>
 <!-- No Preview Modal -->
-{{ widgets.NoPreviewModal(course=course) }}
+{{ widgets.NoPreviewModal(course=course, membership=membership) }}
 
 <script>
 

lms/lms/widgets/NoPreviewModal.html

@@ -29,7 +29,7 @@
                 <button class="btn btn-primary btn-sm notify-me pull-right" data-course="{{course.name | urlencode}}">
                     {{ _("Notify me when available") }}
                 </button>
-                {% else %}
+                {% elif show_start_learing_cta(course, membership) %}
                 <button class="btn btn-primary btn-sm join-batch pull-right" data-course="{{ course.name | urlencode}}">
                     {{ _("Start Learning") }}
                 </button>

lms/www/courses/index.py

@@ -10,13 +10,13 @@ def get_context(context):
     portal_course_creation = frappe.db.get_single_value("LMS Settings", "portal_course_creation")
     context.show_creators_section = frappe.session.user != "Guest" and \
         (portal_course_creation == "Anyone" or has_course_instructor_role())
-    context.show_review_section = has_course_moderator_role() and frappe.session.user != "Gurst"
+    context.show_review_section = has_course_moderator_role() and frappe.session.user != "Guest"
 
     if context.restriction:
         context.restriction_details = get_restriction_details()
 
     context.metatags = {
-        "title": _("Courses List"),
+        "title": _("Course List"),
         "image": frappe.db.get_single_value("Website Settings", "banner_image"),
         "description": "This page lists all the courses published on our website",
         "keywords": "All Courses, Courses, Learn"