fix: onboarding conditions
This commit is contained in:
@@ -722,12 +722,14 @@ def is_onboarding_complete():
|
|||||||
lesson_created = frappe.db.a_row_exists("Course Lesson")
|
lesson_created = frappe.db.a_row_exists("Course Lesson")
|
||||||
|
|
||||||
if course_created and chapter_created and lesson_created:
|
if course_created and chapter_created and lesson_created:
|
||||||
frappe.db.set_value("LMS Settings", "is_onboarding_complete", 1)
|
frappe.db.set_single_value("LMS Settings", "is_onboarding_complete", 1)
|
||||||
|
|
||||||
return {
|
return {
|
||||||
"is_onboarded": frappe.db.get_single_value("LMS Settings", "is_onboarding_complete"),
|
"is_onboarded": frappe.db.get_single_value("LMS Settings", "is_onboarding_complete"),
|
||||||
"course_created": course_created,
|
"course_created": course_created,
|
||||||
"chapter_created": chapter_created,
|
"chapter_created": chapter_created,
|
||||||
"lesson_created": lesson_created,
|
"lesson_created": lesson_created,
|
||||||
"first_course": frappe.get_all("LMS Course", limit=1, order_by=None, pluck="name"),
|
"first_course": frappe.get_all("LMS Course", limit=1, order_by=None, pluck="name")[0]
|
||||||
|
if course_created
|
||||||
|
else None,
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -1,5 +1,5 @@
|
|||||||
{% set onboarding_status = is_onboarding_complete() %}
|
{% set onboarding_status = is_onboarding_complete() %}
|
||||||
{{ onboarding_status.is_onboarded }}
|
|
||||||
{% if has_course_moderator_role() and not onboarding_status.is_onboarded %}
|
{% if has_course_moderator_role() and not onboarding_status.is_onboarded %}
|
||||||
<div class="onboarding-parent">
|
<div class="onboarding-parent">
|
||||||
<div class="container">
|
<div class="container">
|
||||||
@@ -13,21 +13,21 @@
|
|||||||
<div class="onboarding-steps">
|
<div class="onboarding-steps">
|
||||||
<a class="onboarding-steps-link" href="/courses/new-course/edit">
|
<a class="onboarding-steps-link" href="/courses/new-course/edit">
|
||||||
<svg class="icon icon-md mr-2">
|
<svg class="icon icon-md mr-2">
|
||||||
<use href="{% if course_created %} #icon-green-check-circled {% else %} #icon-disabled-check {% endif %}">
|
<use href="{% if onboarding_status.course_created %} #icon-green-check-circled {% else %} #icon-disabled-check {% endif %}">
|
||||||
</use>
|
</use>
|
||||||
</svg>
|
</svg>
|
||||||
{{ _("Create a Course") }}
|
{{ _("Create a Course") }}
|
||||||
</a>
|
</a>
|
||||||
<a class="onboarding-steps-link" {% if onboarding_status.course_created %} href="/courses/{{ onboarding_status.first_course }}/outline" {% else %} disabled {% endif %}>
|
<a class="onboarding-steps-link" {% if onboarding_status.course_created %} href="/courses/{{ onboarding_status.first_course }}/outline" {% else %} disabled {% endif %}>
|
||||||
<svg class="icon icon-md mr-2">
|
<svg class="icon icon-md mr-2">
|
||||||
<use href="{% if chapter_created %} #icon-green-check-circled {% else %} #icon-disabled-check {% endif %}">
|
<use href="{% if onboarding_status.chapter_created %} #icon-green-check-circled {% else %} #icon-disabled-check {% endif %}">
|
||||||
</use>
|
</use>
|
||||||
</svg>
|
</svg>
|
||||||
{{ _("Add a Chapter") }}
|
{{ _("Add a Chapter") }}
|
||||||
</a>
|
</a>
|
||||||
<a class="onboarding-steps-link" {% if onboarding_status.chapter_created %} href="/courses/{{ onboarding_status.first_course }}/learn/1.1/edit" {% endif %}>
|
<a class="onboarding-steps-link" {% if onboarding_status.chapter_created %} href="/courses/{{ onboarding_status.first_course }}/learn/1.1/edit" {% endif %}>
|
||||||
<svg class="icon icon-md mr-2">
|
<svg class="icon icon-md mr-2">
|
||||||
<use href="{% if lesson_created %} #icon-green-check-circled {% else %} #icon-disabled-check {% endif %}">
|
<use href="{% if onboarding_status.lesson_created %} #icon-green-check-circled {% else %} #icon-disabled-check {% endif %}">
|
||||||
</use>
|
</use>
|
||||||
</svg>
|
</svg>
|
||||||
{{ _("Add a Lesson") }}
|
{{ _("Add a Lesson") }}
|
||||||
|
|||||||
Reference in New Issue
Block a user