eduvia/lms
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: made course filter mandatory for course progress summary report

Browse Source
This commit is contained in:
Jannat Patel
2021-11-01 14:46:16 +05:30
parent 125b8d61d9
commit 43311dfb73
2 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
school/lms/report/course_progress_summary/course_progress_summary.js
View File

@@ -8,7 +8,8 @@ frappe.query_reports["Course Progress Summary"] = {
"fieldname": "course", "fieldname": "course",
"label": __("Course"), "label": __("Course"),
"fieldtype": "Link", "fieldtype": "Link",
"options": "LMS Course" "options": "LMS Course",
"reqd": 1,
} }
] ]
}; };

2
school/lms/widgets/CourseCard.html
View File

@@ -57,7 +57,7 @@
</span> {% endif %} </span> {% endif %}
{% set avg_rating = course.get_average_rating() %} {% set avg_rating = course.get_average_rating() %}
{% if avg_rating %} {% if avg_rating %}
<span class="vertically-center "> <span class="vertically-center">
<img class="icon-background" src="/assets/school/icons/rating.svg" /> <img class="icon-background" src="/assets/school/icons/rating.svg" />
{{ frappe.utils.flt(avg_rating, frappe.get_system_settings("float_precision") or 3) }} {{ frappe.utils.flt(avg_rating, frappe.get_system_settings("float_precision") or 3) }}
</span> </span>