feat: added page to see progress of a batch

It has been replaced by Chapter and Lesson.
Moved the section_parser from lms_topic directory to the lms.

community/hooks.py

@@ -15,11 +15,11 @@ app_license = "AGPL"
 # ------------------
 
 # include js, css files in header of desk.html
-app_include_css = "/assets/community/css/community.css"
+# app_include_css = "/assets/community/css/community.css"
-app_include_js = "/assets/community/js/community.js"
+# app_include_js = "/assets/community/js/community.js"
 
 # include js, css files in header of web template
-web_include_css = "/assets/css/community.css"
+web_include_css = "community.bundle.css"
 # web_include_css = "/assets/community/css/community.css"
 # web_include_js = "/assets/community/js/community.js"
 
@@ -141,12 +141,13 @@ primary_rules = [
     {"from_route": "/hackathons/<hackathon>/<project>", "to_route": "hackathons/project"},
     {"from_route": "/dashboard", "to_route": ""},
     {"from_route": "/add-a-new-batch", "to_route": "add-a-new-batch"},
-    {"from_route": "/courses/<course>/<batch>/learn", "to_route": "courses/learn"},
+    {"from_route": "/courses/<course>/<batch>/learn", "to_route": "batch/learn"},
-    {"from_route": "/courses/<course>/<batch>/learn/<int:chapter>.<int:lesson>", "to_route": "courses/learn"},
+    {"from_route": "/courses/<course>/<batch>/learn/<int:chapter>.<int:lesson>", "to_route": "batch/learn"},
-    {"from_route": "/courses/<course>/<batch>/schedule", "to_route": "courses/schedule"},
+    {"from_route": "/courses/<course>/<batch>/schedule", "to_route": "batch/schedule"},
-    {"from_route": "/courses/<course>/<batch>/members", "to_route": "courses/members"},
+    {"from_route": "/courses/<course>/<batch>/members", "to_route": "batch/members"},
-    {"from_route": "/courses/<course>/<batch>/discuss", "to_route": "courses/discuss"},
+    {"from_route": "/courses/<course>/<batch>/discuss", "to_route": "batch/discuss"},
-    {"from_route": "/courses/<course>/<batch>/about", "to_route": "courses/about"}
+    {"from_route": "/courses/<course>/<batch>/about", "to_route": "batch/about"},
+    {"from_route": "/courses/<course>/<batch>/progress", "to_route": "batch/progress"}
 ]
 
 # Any frappe default URL is blocked by profile-rules, add it here to unblock it

community/lms/api.py

@@ -21,3 +21,16 @@ def get_section(name):
     """
     doc = frappe.get_doc("LMS Section", name)
     return doc and doc.as_dict()
+
+@frappe.whitelist()
+def submit_solution(exercise, code):
+    """Submits a solution.
+
+    @exerecise: name of the exercise to submit
+    @code: solution to the exercise
+    """
+    ex = frappe.get_doc("Exercise", exercise)
+    if not ex:
+        return
+    doc = ex.submit(code)
+    return {"name": doc.name, "creation": doc.creation}

community/lms/doctype/exercise/exercise.js

@@ -1,7 +1,7 @@
 // Copyright (c) 2021, FOSS United and contributors
 // For license information, please see license.txt
 
-frappe.ui.form.on('LMS Topic', {
+frappe.ui.form.on('Exercise', {
 	// refresh: function(frm) {
 
 	// }

community/lms/doctype/exercise/exercise.json

@@ -0,0 +1,106 @@
+{
+ "actions": [],
+ "allow_rename": 1,
+ "creation": "2021-05-19 17:43:39.923430",
+ "doctype": "DocType",
+ "editable_grid": 1,
+ "engine": "InnoDB",
+ "field_order": [
+  "title",
+  "description",
+  "code",
+  "answer",
+  "column_break_4",
+  "course",
+  "hints",
+  "tests",
+  "image",
+  "lesson"
+ ],
+ "fields": [
+  {
+   "fieldname": "title",
+   "fieldtype": "Data",
+   "in_list_view": 1,
+   "label": "Title"
+  },
+  {
+   "fieldname": "course",
+   "fieldtype": "Link",
+   "label": "Course",
+   "options": "LMS Course"
+  },
+  {
+   "columns": 4,
+   "fieldname": "description",
+   "fieldtype": "Small Text",
+   "in_list_view": 1,
+   "label": "Description"
+  },
+  {
+   "columns": 4,
+   "fieldname": "answer",
+   "fieldtype": "Code",
+   "label": "Answer"
+  },
+  {
+   "fieldname": "tests",
+   "fieldtype": "Code",
+   "label": "Tests"
+  },
+  {
+   "fieldname": "column_break_4",
+   "fieldtype": "Column Break"
+  },
+  {
+   "columns": 4,
+   "fieldname": "hints",
+   "fieldtype": "Small Text",
+   "label": "Hints"
+  },
+  {
+   "columns": 4,
+   "fieldname": "code",
+   "fieldtype": "Code",
+   "label": "Code"
+  },
+  {
+   "fieldname": "image",
+   "fieldtype": "Code",
+   "label": "Image",
+   "read_only": 1
+  },
+  {
+   "fieldname": "lesson",
+   "fieldtype": "Link",
+   "label": "Lesson",
+   "options": "Lesson"
+  }
+ ],
+ "index_web_pages_for_search": 1,
+ "links": [],
+ "modified": "2021-05-20 13:23:12.340928",
+ "modified_by": "Administrator",
+ "module": "LMS",
+ "name": "Exercise",
+ "owner": "Administrator",
+ "permissions": [
+  {
+   "create": 1,
+   "delete": 1,
+   "email": 1,
+   "export": 1,
+   "print": 1,
+   "read": 1,
+   "report": 1,
+   "role": "System Manager",
+   "share": 1,
+   "write": 1
+  }
+ ],
+ "search_fields": "title",
+ "sort_field": "modified",
+ "sort_order": "DESC",
+ "title_field": "title",
+ "track_changes": 1
+}

community/lms/doctype/exercise/exercise.py

@@ -0,0 +1,59 @@
+# Copyright (c) 2021, FOSS United and contributors
+# For license information, please see license.txt
+
+import frappe
+from frappe.model.document import Document
+from ..lms_sketch.livecode import livecode_to_svg
+
+class Exercise(Document):
+    def before_save(self):
+        self.image = livecode_to_svg(None, self.answer)
+
+    def get_user_submission(self):
+        """Returns the latest submission for this user.
+        """
+        user = frappe.session.user
+        if not user or user == "Guest":
+            return
+
+        result = frappe.get_all('Exercise Submission',
+            fields="*",
+            filters={
+                "owner": user,
+                "exercise": self.name
+            },
+            order_by="creation desc",
+            page_length=1)
+
+        print("get_user_submission", result)
+        if result:
+            return result[0]
+
+    def submit(self, code):
+        """Submits the given code as solution to exercise.
+        """
+        user = frappe.session.user
+        if not user or user == "Guest":
+            return
+
+        old_submission = self.get_user_submission()
+        if old_submission and old_submission.solution == code:
+            return old_submission
+
+        course = frappe.get_doc("LMS Course", self.course)
+        batch = course.get_student_batch(user)
+
+        image = livecode_to_svg(None, code)
+
+        doc = frappe.get_doc(
+            doctype="Exercise Submission",
+            exercise=self.name,
+            exercise_title=self.title,
+            course=self.course,
+            lesson=self.lesson,
+            batch=batch and batch.name,
+            image=image,
+            solution=code)
+        doc.insert(ignore_permissions=True)
+        return doc
+

community/lms/doctype/exercise/test_exercise.py

@@ -0,0 +1,47 @@
+# Copyright (c) 2021, FOSS United and Contributors
+# See license.txt
+
+import frappe
+import unittest
+
+class TestExercise(unittest.TestCase):
+    def setUp(self):
+        frappe.db.sql('delete from `tabExercise Submission`')
+        frappe.db.sql('delete from `tabExercise`')
+        frappe.db.sql('delete from `tabLMS Course`')
+
+    def new_exercise(self):
+        course = frappe.get_doc({
+            "doctype": "LMS Course",
+            "name": "test-course",
+            "title": "Test Course"
+        })
+        course.insert()
+        e = frappe.get_doc({
+            "doctype": "Exercise",
+            "name": "test-problem",
+            "course": course.name,
+            "title": "Test Problem",
+            "description": "draw a circle",
+            "code": "# draw a single cicle",
+            "answer": (
+                "# draw a single circle\n" +
+                "circle(100, 100, 50)")
+        })
+        e.insert()
+        return e
+
+    def test_exercise(self):
+        e = self.new_exercise()
+        assert e.get_user_submission() is None
+
+    def test_exercise_submission(self):
+        e = self.new_exercise()
+        submission = e.submit("circle(100, 100, 50)")
+        assert submission is not None
+        assert submission.exercise == e.name
+        assert submission.course == e.course
+
+        user_submission = e.get_user_submission()
+        assert user_submission is not None
+        assert user_submission.name == submission.name

community/lms/doctype/exercise_submission/exercise_submission.js

@@ -0,0 +1,8 @@
+// Copyright (c) 2021, FOSS United and contributors
+// For license information, please see license.txt
+
+frappe.ui.form.on('Exercise Submission', {
+	// refresh: function(frm) {
+
+	// }
+});

community/lms/doctype/exercise_submission/exercise_submission.json

@@ -0,0 +1,90 @@
+{
+ "actions": [],
+ "creation": "2021-05-19 11:41:18.108316",
+ "doctype": "DocType",
+ "editable_grid": 1,
+ "engine": "InnoDB",
+ "field_order": [
+  "exercise",
+  "solution",
+  "exercise_title",
+  "course",
+  "batch",
+  "lesson",
+  "image"
+ ],
+ "fields": [
+  {
+   "fieldname": "exercise",
+   "fieldtype": "Link",
+   "in_list_view": 1,
+   "label": "Exercise",
+   "options": "Exercise"
+  },
+  {
+   "fieldname": "solution",
+   "fieldtype": "Code",
+   "in_list_view": 1,
+   "label": "Solution"
+  },
+  {
+   "fetch_from": "exercise.title",
+   "fieldname": "exercise_title",
+   "fieldtype": "Data",
+   "in_list_view": 1,
+   "label": "Exercise Title",
+   "read_only": 1
+  },
+  {
+   "fetch_from": "exercise.course",
+   "fieldname": "course",
+   "fieldtype": "Link",
+   "label": "Course",
+   "options": "LMS Course",
+   "read_only": 1
+  },
+  {
+   "fieldname": "batch",
+   "fieldtype": "Link",
+   "label": "Batch",
+   "options": "LMS Batch"
+  },
+  {
+   "fetch_from": "exercise.lesson",
+   "fieldname": "lesson",
+   "fieldtype": "Link",
+   "label": "Lesson",
+   "options": "Lesson"
+  },
+  {
+   "fieldname": "image",
+   "fieldtype": "Code",
+   "label": "Image",
+   "read_only": 1
+  }
+ ],
+ "index_web_pages_for_search": 1,
+ "links": [],
+ "modified": "2021-05-21 11:28:45.833018",
+ "modified_by": "Administrator",
+ "module": "LMS",
+ "name": "Exercise Submission",
+ "owner": "Administrator",
+ "permissions": [
+  {
+   "create": 1,
+   "delete": 1,
+   "email": 1,
+   "export": 1,
+   "print": 1,
+   "read": 1,
+   "report": 1,
+   "role": "System Manager",
+   "share": 1,
+   "write": 1
+  }
+ ],
+ "sort_field": "modified",
+ "sort_order": "DESC",
+ "track_changes": 1
+}

community/lms/doctype/exercise_submission/exercise_submission.py

@@ -0,0 +1,8 @@
+# Copyright (c) 2021, FOSS United and contributors
+# For license information, please see license.txt
+
+# import frappe
+from frappe.model.document import Document
+
+class ExerciseSubmission(Document):
+	pass

community/lms/doctype/exercise_submission/test_exercise_submission.py

@@ -1,10 +1,8 @@
-# -*- coding: utf-8 -*-
 # Copyright (c) 2021, FOSS United and Contributors
 # See license.txt
-from __future__ import unicode_literals
 
 # import frappe
 import unittest
 
-class TestLMSTopic(unittest.TestCase):
+class TestExerciseSubmission(unittest.TestCase):
 	pass

community/lms/doctype/lesson/lesson.py

@@ -5,12 +5,17 @@
 from __future__ import unicode_literals
 import frappe
 from frappe.model.document import Document
-from ..lms_topic.section_parser import SectionParser
+from ...section_parser import SectionParser
 
 class Lesson(Document):
     def before_save(self):
         sections = SectionParser().parse(self.body or "")
         self.sections = [self.make_lms_section(i, s) for i, s in enumerate(sections)]
+        for s in self.sections:
+            if s.type == "exercise":
+                e = s.get_exercise()
+                e.lesson = self.name
+                e.save()
 
     def get_sections(self):
         return sorted(self.get('sections'), key=lambda s: s.index)
@@ -18,6 +23,7 @@ class Lesson(Document):
     def make_lms_section(self, index, section):
             s = frappe.new_doc('LMS Section', parent_doc=self, parentfield='sections')
             s.type = section.type
+            s.id = section.id
             s.label = section.label
             s.contents = section.contents
             s.index = index

community/lms/doctype/lms_batch/lms_batch.py

@@ -37,13 +37,37 @@ class LMSBatch(Document):
         member_names = [m['member'] for m in memberships]
         return find_all("Community Member", name=["IN", member_names])
 
-    def is_member(self, email):
+    def is_member(self, email, member_type=None):
         """Checks if a person is part of a batch.
+
+        If member_type is specified, checks if the person is a Student/Mentor.
         """
         member = find("Community Member", email=email)
-        return member and frappe.db.exists(
+        if not member:
+            return
+
+        filters = {
+            "batch": self.name,
+            "member": member.name
+        }
+        if member_type:
+            filters['member_type'] = member_type
+        return frappe.db.exists("LMS Batch Membership", filters)
+
+    def get_students(self):
+        """Returns (email, full_name, username) of all the students of this batch as a list of dict.
+        """
+        memberships = frappe.get_all(
                     "LMS Batch Membership",
-                    {"batch": self.name, "member": member.name})
+                    {"batch": self.name, "member_type": "Student"},
+                    ["member"])
+        member_names = [m['member'] for m in memberships]
+        members = frappe.get_all(
+                    "Community Member",
+                    {"name": ["IN", member_names]},
+                    ["email", "full_name", "username"])
+        return members
+
 
 @frappe.whitelist()
 def get_messages(batch):

community/lms/doctype/lms_course/lms_course.py

@@ -35,17 +35,6 @@ class LMSCourse(Document):
     def __repr__(self):
         return f"<Course#{self.name}>"
 
-    def get_topic(self, slug):
-        """Returns the topic with given slug in this course as a Document.
-        """
-        result = frappe.get_all(
-            "LMS Topic",
-            filters={"course": self.name, "slug": slug})
-
-        if result:
-            row = result[0]
-            return frappe.get_doc('LMS Topic', row['name'])
-
     def has_mentor(self, email):
         """Checks if this course has a mentor with given email.
         """

community/lms/doctype/lms_section/lms_section.json

@@ -9,7 +9,8 @@
   "contents",
   "code",
   "attrs",
-  "index"
+  "index",
+  "id"
  ],
  "fields": [
   {
@@ -43,12 +44,17 @@
    "fieldname": "index",
    "fieldtype": "Int",
    "label": "Index"
+  },
+  {
+   "fieldname": "id",
+   "fieldtype": "Data",
+   "label": "id"
   }
  ],
  "index_web_pages_for_search": 1,
  "istable": 1,
  "links": [],
- "modified": "2021-03-12 17:56:23.118854",
+ "modified": "2021-05-19 18:55:26.019625",
  "modified_by": "Administrator",
  "module": "LMS",
  "name": "LMS Section",

community/lms/doctype/lms_section/lms_section.py

@@ -10,6 +10,10 @@ class LMSSection(Document):
     def __repr__(self):
         return f"<LMSSection {self.label!r}>"
 
+    def get_exercise(self):
+        if self.type == "exercise":
+            return frappe.get_doc("Exercise", self.id)
+
     def get_latest_code_for_user(self):
         """Returns the latest code for the logged in user.
         """

community/lms/doctype/lms_sketch/livecode.py

@@ -4,6 +4,7 @@ import websocket
 import json
 from .svg import SVG
 import frappe
+from urllib.parse import urlparse
 
 # Files to pass to livecode server
 # The same code is part of livecode-canvas.js
@@ -60,9 +61,21 @@ def clear():
 clear()
 '''
 
+def get_livecode_url():
+    doc = frappe.get_cached_doc("LMS Settings")
+    return doc.livecode_url
+
+def get_livecode_ws_url():
+    url = urlparse(get_livecode_url())
+    protocol = "wss" if url.scheme == "https" else "ws"
+    return protocol + "://" + url.netloc + "/livecode"
+
 def livecode_to_svg(livecode_ws_url, code, *, timeout=3):
     """Renders the code as svg.
     """
+    if livecode_ws_url is None:
+        livecode_ws_url = get_livecode_ws_url()
+
     try:
         ws = websocket.WebSocket()
         ws.settimeout(timeout)

community/lms/doctype/lms_topic/lms_topic.json

@@ -1,89 +0,0 @@
-{
- "actions": [],
- "allow_guest_to_view": 1,
- "creation": "2021-03-02 07:20:41.686573",
- "doctype": "DocType",
- "editable_grid": 1,
- "engine": "InnoDB",
- "field_order": [
-  "course",
-  "title",
-  "slug",
-  "preview",
-  "description",
-  "order",
-  "sections"
- ],
- "fields": [
-  {
-   "fieldname": "title",
-   "fieldtype": "Data",
-   "in_list_view": 1,
-   "label": "Title",
-   "reqd": 1
-  },
-  {
-   "fieldname": "description",
-   "fieldtype": "Markdown Editor",
-   "label": "Description"
-  },
-  {
-   "fieldname": "course",
-   "fieldtype": "Link",
-   "in_list_view": 1,
-   "label": "Course",
-   "options": "LMS Course",
-   "reqd": 1
-  },
-  {
-   "fieldname": "order",
-   "fieldtype": "Int",
-   "label": "Order"
-  },
-  {
-   "fieldname": "preview",
-   "fieldtype": "Markdown Editor",
-   "label": "Preview"
-  },
-  {
-   "fieldname": "sections",
-   "fieldtype": "Table",
-   "label": "Sections",
-   "options": "LMS Section"
-  },
-  {
-   "description": "The slug of the topic. Autogenerated from the title if not specified.",
-   "fieldname": "slug",
-   "fieldtype": "Data",
-   "label": "Slug"
-  }
- ],
- "index_web_pages_for_search": 1,
- "links": [],
- "modified": "2021-04-06 14:12:48.514062",
- "modified_by": "Administrator",
- "module": "LMS",
- "name": "LMS Topic",
- "owner": "Administrator",
- "permissions": [
-  {
-   "create": 1,
-   "delete": 1,
-   "email": 1,
-   "export": 1,
-   "print": 1,
-   "read": 1,
-   "report": 1,
-   "role": "System Manager",
-   "share": 1,
-   "write": 1
-  }
- ],
- "search_fields": "title",
- "sort_field": "creation",
- "sort_order": "ASC",
- "title_field": "title",
- "track_changes": 1,
- "track_seen": 1,
- "track_views": 1
-}

community/lms/doctype/lms_topic/lms_topic.py

@@ -1,43 +0,0 @@
-# -*- coding: utf-8 -*-
-# Copyright (c) 2021, FOSS United and contributors
-# For license information, please see license.txt
-
-from __future__ import unicode_literals
-import frappe
-from frappe.model.document import Document
-from .section_parser import SectionParser
-from ...utils import slugify
-
-class LMSTopic(Document):
-    def before_save(self):
-        course = self.get_course()
-        if not self.slug:
-            self.slug = self.generate_slug(title=self.title)
-
-        sections = SectionParser().parse(self.description or "")
-        self.sections = [self.make_lms_section(i, s) for i, s in enumerate(sections)]
-
-    def get_course(self):
-        return frappe.get_doc("LMS Course", self.course)
-
-    def generate_slug(self, title):
-        result = frappe.get_all(
-            'LMS Topic',
-            filters={'course': self.course},
-            fields=['slug'])
-        slugs = set([row['slug'] for row in result])
-        return slugify(title, used_slugs=slugs)
-
-    def get_sections(self):
-        return sorted(self.sections, key=lambda s: s.index)
-
-    def make_lms_section(self, index, section):
-            s = frappe.new_doc('LMS Section', parent_doc=self, parentfield='sections')
-            s.type = section.type
-            s.label = section.label
-            s.contents = section.contents
-            s.index = index
-            return s
-
-
-

community/lms/widgets/Exercise.html

@@ -0,0 +1,18 @@
+{% from "www/macros/livecode.html" import LiveCodeEditorJS, LiveCodeEditor with context %}
+
+<div class="exercise">
+  <h2>{{ exercise.title }}</h2>
+  <div class="exercise-description">{{frappe.utils.md_to_html(exercise.description)}}</div>
+
+  {% if exercise.image %}
+  <div class="exercise-image">{{exercise.image}}</div>
+  {% endif %}
+
+  {% set submission = exercise.get_user_submission() %}
+
+  {{ LiveCodeEditor(exercise.name,
+        code=submission.solution if submission else exercise.code,
+        reset_code=exercise.code,
+        is_exercise=True,
+        last_submitted=submission and submission.creation) }}
+</div>

community/public/build.json

@@ -1,14 +0,0 @@
-{
-  "css/lms.css": [
-    "public/css/lms.css"
-  ],
-  "css/community.css": [
-    "public/css/style.css",
-    "public/css/vars.css",
-    "public/css/style.less"
-  ],
-  "js/livecode-canvas.js": [
-    "public/js/livecode-canvas.js"
-  ]
-
-}

community/public/css/community.bundle.less

@@ -0,0 +1,4 @@
+
+@import "./style.css";
+@import "./vars.css";
+@import "./style.less";

community/public/css/lms.css

@@ -1,26 +0,0 @@
-
-.heading {
-  background: #eee;
-  padding: 10px;
-  clear: both;
-  color: #212529;
-  border: 1px solid #ddd;
-}
-
-.sketch-header h1 {
-  font-size: 1.5em;
-  margin-bottom: 0px;
-}
-
-.sketch-header {
-  margin-bottom: 1em;
-}
-
-.sketch-card .sketch-title a {
-  font-weight: bold;
-  color: inherit;
-}
-
-.hidden {
-  display: none;
-}

community/public/css/style.css

@@ -1,5 +1,3 @@
-@import url("https://cdn.jsdelivr.net/npm/bootstrap-icons@1.4.1/font/bootstrap-icons.css");
-@import url("https://use.fontawesome.com/releases/v5.13.0/css/all.css");
 
 :root {
   --c1: #fefae0;
@@ -26,7 +24,7 @@
   --send-message: var(--c7);
   --received-message: var(--c8);
 
-  --primary-color: #08B74F;
+  --primary-color: #08B74F !important;
 }
 
 body {

community/public/css/style.less

@@ -1,9 +1,4 @@
 @primary-color: #08B74F;
-@import url('https://rsms.me/inter/inter.css');
-
-body {
-  font-family: "Inter", sans-serif;
-}
 
 h2 {
   margin: 20px 0px;
@@ -306,3 +301,39 @@ section.lightgray {
 .lesson-page {
   margin: 20px 0px;
 }
+
+.lesson-pagination {
+  clear: both;
+}
+
+.exercise-image svg {
+  width: 200px;
+  height: 200px;
+  border: 1px solid #ddd;
+  margin-bottom: 20px;
+}
+
+.svg-200 svg {
+  width: 200px;
+  height: 200px;
+}
+
+.livecode-editor-small .livecode-editor {
+  .CodeMirror-scroll {
+    max-height: 160px;
+    min-height: 160px;
+  }
+
+  canvas {
+    width: 150px;
+    height: 150px;
+  }
+}
+
+.mentor-dashboard {
+  margin-top: 20px;
+
+  .submission {
+    margin: 40px 0px 0px 20px;
+  }
+}

community/www/batch/about.html

@@ -6,10 +6,13 @@
 {% block head_include %}
 <meta name="description" content="Courses" />
 <meta name="keywords" content="" />
+
+<link rel="stylesheet" href="/assets/frappe/css/font-awesome.css">
 {% endblock %}
 
 {% block content %}
-{{ Sidebar(course.name, batch.name) }}
+{{ Sidebar(course, batch) }}
+
 <div class="container">
 	{{ CourseBasicDetail(course)}}
 	{{ InstructorsSection(course.get_instructor()) }}

community/www/batch/about.py

@@ -0,0 +1,7 @@
+import frappe
+from . import utils
+
+def get_context(context):
+    utils.get_common_context(context)
+
+    print("context", context)

community/www/batch/discuss.html

@@ -6,13 +6,16 @@
 {% block head_include %}
 <meta name="description" content="Courses" />
 <meta name="keywords" content="" />
+
+<link rel="stylesheet" href="/assets/frappe/css/font-awesome.css">
 {% endblock %}
 
 {% block content %}
-{{ Sidebar(course_slug, batch_code) }}
+{{ Sidebar(course, batch) }}
+
 <div class="">
   <div class="batch-header">
-    {{ BatchHearder(course.name, member_count) }}
+    {{ BatchHearder(course.title, member_count) }}
   </div>
   <div class="messages">
     <div class="message-section">

community/www/batch/discuss.py

@@ -0,0 +1,9 @@
+import frappe
+from . import utils
+from community.lms.doctype.lms_batch.lms_batch import get_messages
+
+def get_context(context):
+    utils.get_common_context(context)
+
+    context.members = utils.get_batch_members(context.batch.name)
+    context.member_count = len(context.members)

community/www/batch/learn.html

@@ -9,6 +9,7 @@
 	<style>
 	</style>
 
+<link rel="stylesheet" href="/assets/frappe/css/font-awesome.css">
 <link rel="stylesheet" href="{{ livecode_url }}/static/codemirror/lib/codemirror.css">
 <link rel="stylesheet" href="/assets/css/lms.css">
 
@@ -22,18 +23,11 @@
 
 
 {% block content %}
-{{ Sidebar(course.name, batch.name) }}
+{{ Sidebar(course, batch) }}
 
 <div class="container">
   <div class="lesson-page">
-    <div class="lesson-pagination">
+    {{ pagination(prev_url, next_url) }}
-      {% if prev_url %}
-        <a href="{{prev_url}}" class="btn">&larr; Prev</a>
-      {% endif %}
-      {% if next_url %}
-        <a href="{{next_url}}" class="btn pull-right">Next &rarr;</a>
-      {% endif %}
-    </div>
 
     <h2>{{ lesson.title }}</h2>
 
@@ -43,10 +37,8 @@
     </div>
     {% endfor %}
 
-    <div class="lesson-pagination">
+    {{ pagination(prev_url, next_url) }}
-      <a href="#" class="btn">&larr; Prev</a>
+
-      <a href="#" class="btn pull-right">Next &rarr;</a>
-    </div>
   </div>
 </div>
 {% endblock %}
@@ -55,8 +47,14 @@
 {% macro render_section(s) %}
   {% if s.type == "text" %}
     {{ render_section_text(s) }}
-  {% elif s.type == "example" or s.type == "code" or s.type == "exercise" %}
+  {% elif s.type == "example" or s.type == "code" %}
-    {{ LiveCodeEditor(s.name, s.get_latest_code_for_user(), s.type=="exercise", "2 hours ago") }}
+    {{ LiveCodeEditor(s.name,
+          code=s.get_latest_code_for_user(),
+          reset_code=s.contents,
+          is_exercise=False)
+    }}
+  {% elif s.type == "exercise" %}
+    {{ widgets.Exercise(exercise=s.get_exercise())}}
   {% else %}
     <div>Unknown section type: {{s.type}}</div>
   {% endif %}
@@ -70,6 +68,18 @@
   </div>
 {% endmacro %}
 
+{% macro pagination(prev_url, next_url) %}
+  <div class="lesson-pagination">
+    {% if prev_url %}
+      <a href="{{prev_url}}" class="btn">&larr; Prev</a>
+    {% endif %}
+    {% if next_url %}
+      <a href="{{next_url}}" class="btn pull-right">Next &rarr;</a>
+    {% endif %}
+    <div style="clear: both;"></div>
+  </div>
+{% endmacro %}
+
 {%- block script %}
   {{ super() }}
   {{ LiveCodeEditorJS() }}

community/www/batch/learn.py

@@ -1,37 +1,25 @@
 import frappe
-from community.lms.models import Course
+from . import utils
 
 def get_context(context):
-    context.no_cache = 1
+    utils.get_common_context(context)
 
-    course_name = frappe.form_dict["course"]
-    batch_name = frappe.form_dict["batch"]
     chapter_index = frappe.form_dict.get("chapter")
     lesson_index = frappe.form_dict.get("lesson")
     lesson_number = f"{chapter_index}.{lesson_index}"
 
-    course = Course.find(course_name)
+    course_name = context.course.name
-    if not course:
+    batch_name = context.batch.name
-        context.template = "www/404.html"
-        return
-
-    batch = course.get_batch(batch_name)
-    if not batch:
-        frappe.local.flags.redirect_location = "/courses/" + course_name
-        raise frappe.Redirect
 
     if not chapter_index or not lesson_index:
         frappe.local.flags.redirect_location = f"/courses/{course_name}/{batch_name}/learn/1.1"
         raise frappe.Redirect
 
-    context.course = course
+    context.lesson = context.course.get_lesson(chapter_index, lesson_index)
-    context.batch = batch
-    context.lesson = course.get_lesson(chapter_index, lesson_index)
     context.lesson_index = lesson_index
     context.chapter_index = chapter_index
-    context.livecode_url = get_livecode_url()
 
-    outline = course.get_outline()
+    outline = context.course.get_outline()
     next_ = outline.get_next(lesson_number)
     prev_ = outline.get_prev(lesson_number)
     context.next_url = get_learn_url(course_name, batch_name, next_)
@@ -41,6 +29,3 @@ def get_learn_url(course_name, batch_name, lesson_number):
     if not lesson_number:
         return
     return f"/courses/{course_name}/{batch_name}/learn/{lesson_number}"
-
-def get_livecode_url():
-    return frappe.db.get_single_value("LMS Settings", "livecode_url")

community/www/batch/members.html

@@ -7,12 +7,14 @@
 {% block head_include %}
 <meta name="description" content="Courses" />
 <meta name="keywords" content="" />
+<link rel="stylesheet" href="/assets/frappe/css/font-awesome.css">
 {% endblock %}
 
 {% block content %}
-{{ Sidebar(course_slug, batch_code) }}
+{{ Sidebar(course, batch) }}
+
 <div class="container">
-	{{ BatchHearder(course.name, member_count)}}
+	{{ BatchHearder(course.title, member_count)}}
 	{{ MembersList(members)}}
 </div>
 {% endblock %}

community/www/batch/members.py

@@ -0,0 +1,8 @@
+import frappe
+from . import utils
+
+def get_context(context):
+    utils.get_common_context(context)
+
+    context.members = utils.get_batch_members(context.batch.name)
+    context.member_count = len(context.members)

community/www/batch/progress.html

@@ -0,0 +1,52 @@
+{% extends "templates/base.html" %}
+{% from "www/macros/sidebar.html" import Sidebar %}
+{% from "www/macros/livecode.html" import LiveCodeEditorJS, LiveCodeEditor with context %}
+{% block title %}{{ course.title  }} - Batch Dashboard{% endblock %}
+
+{% block head_include %}
+	<meta name="description" content="{{course.title}} - Batch Dashboard" />
+	<meta name="keywords" content="{{course.title}} - Batch Dashboard" />
+	<style>
+	</style>
+
+<link rel="stylesheet" href="/assets/frappe/css/font-awesome.css">
+<link rel="stylesheet" href="{{ livecode_url }}/static/codemirror/lib/codemirror.css">
+<link rel="stylesheet" href="/assets/css/lms.css">
+
+<script src="{{ livecode_url }}/static/codemirror/lib/codemirror.js"></script>
+<script src="{{ livecode_url }}/static/codemirror/mode/python/python.js"></script>
+<script src="{{ livecode_url }}/static/codemirror/keymap/sublime.js"></script>
+
+<script src="{{ livecode_url }}/static/codemirror/addon/edit/matchbrackets.js"></script>
+<script src="{{ livecode_url }}/static/codemirror/addon/comment/comment.js"></script>
+{% endblock %}
+
+
+{% block content %}
+{{ Sidebar(course, batch) }}
+
+<div class="container">
+  <div class="mentor-dashboard">
+    <h1>Batch Progress</h1>
+    {% for exercise in report.exercises %}
+      <div class="exercise-submissions">
+        <h2>{{exercise.title}}</h2>
+        {% for s in report.get_submissions_of_exercise(exercise.name) %}
+          <div class="submission">
+            <h4><a href="/{{s.owner.username}}">{{s.owner.full_name}}</a></h4>
+            <div class="livecode-editor-small">
+              {{ LiveCodeEditor(name=s.name, code=s.solution, reset_code=s.solution) }}
+            </div>
+          </div>
+        {% endfor %}
+      </div>
+    {% endfor %}
+  </div>
+</div>
+{% endblock %}
+
+
+{%- block script %}
+  {{ super() }}
+  {{ LiveCodeEditorJS() }}
+{% endblock %}

community/www/batch/progress.py

@@ -0,0 +1,64 @@
+import frappe
+from community.lms.models import Course
+from collections import defaultdict
+from . import utils
+
+def get_context(context):
+    utils.get_common_context(context)
+
+    exercise_name = frappe.form_dict.get("exercise")
+    if exercise_name:
+        exercise = frappe.get_doc("Exercise", exercise_name)
+    else:
+        exercise = None
+
+    context.exercise = exercise
+    context.report = BatchReport(context.course, context.batch)
+
+class BatchReport:
+    def __init__(self, course, batch):
+        self.submissions = get_submissions(batch)
+        self.exercises = self.get_exercises(course.name)
+
+        self.submissions_by_exercise = defaultdict(list)
+        for s in self.submissions:
+            self.submissions_by_exercise[s.exercise].append(s)
+
+    def get_exercises(self, course_name):
+        return frappe.get_all("Exercise", {"course": course_name}, ["name", "title"])
+
+    def get_submissions_of_exercise(self, exercise_name):
+        return self.submissions_by_exercise[exercise_name]
+
+def get_submissions(batch):
+    students = batch.get_students()
+    students_map = {s['email']: s for s in students}
+
+    names, values = nparams("s", students_map.keys())
+
+    sql = """
+    select owner, exercise, name, solution, creation, image
+    from (
+        select owner, exercise, name, solution, creation, image,
+            row_number() over (partition by owner, exercise order by creation desc) as ix
+        from `tabExercise Submission`) as t
+    where t.ix=1 and owner IN {}
+    """.format(names)
+
+    data = frappe.db.sql(sql, values=values, as_dict=True)
+
+    for row in data:
+        row['owner'] = students_map[row['owner']]
+    return data
+
+def nparams(name, values):
+    """Creates n paramters from a list of values for a db query.
+
+    >>> nparams("name", ["a", "b])
+    ("(%(name_1)s, %(name_2)s)", {"name_1": "a", "name_2": "b"})
+    """
+    keys = [f"{name}_{i}" for i, _ in enumerate(values, start=1)]
+    param_names = [f"%({k})s" for k in keys]
+    param_values = dict(zip(keys, values))
+    joined_names = "(" + ", ".join(param_names) + ")"
+    return joined_names, param_values

community/www/batch/schedule.html

@@ -4,10 +4,11 @@
 {% block head_include %}
 <meta name="description" content="Courses" />
 <meta name="keywords" content="" />
+<link rel="stylesheet" href="/assets/frappe/css/font-awesome.css">
 {% endblock %}
 
 {% block content %}
-{{ Sidebar(course, batch_code) }}
+{{ Sidebar(course, batch) }}
 <div class="container">
 </div>
 {% endblock %}

community/www/batch/schedule.py

@@ -3,7 +3,6 @@ from community.lms.models import Course
 
 def get_context(context):
     context.no_cache = 1
-
     course_name = frappe.form_dict["course"]
     batch_name = frappe.form_dict["batch"]
 

community/www/batch/utils.py

@@ -0,0 +1,43 @@
+import frappe
+from community.lms.models import Course
+
+def get_common_context(context):
+    context.no_cache = 1
+
+    course_name = frappe.form_dict["course"]
+    batch_name = frappe.form_dict["batch"]
+
+    course = Course.find(course_name)
+    if not course:
+        context.template = "www/404.html"
+        return
+
+    batch = course.get_batch(batch_name)
+    if not batch:
+        frappe.local.flags.redirect_location = "/courses/" + course_name
+        raise frappe.Redirect
+
+    context.course = course
+    context.batch = batch
+    context.livecode_url = get_livecode_url()
+
+def get_livecode_url():
+    return frappe.db.get_single_value("LMS Settings", "livecode_url")
+
+def get_batch_members(batch_name):
+    members = []
+    memberships = frappe.get_all("LMS Batch Membership", {"batch": batch_name}, ["member", "member_type"])
+
+    for membership in memberships:
+        member = get_member_with_name(membership.member)
+        if membership.member_type == "Mentor":
+            member.is_mentor = True
+        members.append(member)
+    return members
+
+def get_member_with_name(name):
+    try:
+        return frappe.get_doc("Community Member", name)
+    except frappe.DoesNotExistError:
+        return
+

community/www/courses/discuss/index.py

@@ -1,15 +0,0 @@
-import frappe
-from community.www.courses.utils import redirect_if_not_a_member, get_course, get_batch_members, get_batch
-from community.lms.doctype.lms_batch.lms_batch import get_messages
-
-def get_context(context):
-    context.no_cache = 1
-    context.course_slug = frappe.form_dict["course"]
-    context.course = get_course(context.course_slug)
-    context.batch_code = frappe.form_dict["batch"]
-    redirect_if_not_a_member(context.course_slug, context.batch_code)
-    
-    context.batch = get_batch(context.batch_code)
-    context.members = get_batch_members(context.batch.name)
-    context.member_count = len(context.members)
-    context.messages = get_messages(context.batch.name)

community/www/courses/index.py

@@ -7,6 +7,6 @@ def get_context(context):
 def get_courses():
     courses = frappe.get_all(
         "LMS Course",
-        fields=['name', 'slug', 'title', 'description']
+        fields=['name', 'title', 'description']
     )
     return courses

community/www/courses/members/index.py

@@ -1,12 +0,0 @@
-import frappe
-from community.www.courses.utils import redirect_if_not_a_member, get_batch, get_member_with_name, get_course, get_batch_members
-
-def get_context(context):
-    context.no_cache = 1
-    context.course_slug = frappe.form_dict["course"]
-    context.course = get_course(context.course_slug)
-    context.batch_code = frappe.form_dict["batch"]
-    redirect_if_not_a_member(context.course_slug, context.batch_code)
-    context.batch = get_batch(context.batch_code)
-    context.members = get_batch_members(context.batch.name)
-    context.member_count = len(context.members)

community/www/courses/schedule/index.py

@@ -1,8 +0,0 @@
-import frappe
-from community.www.courses.utils import redirect_if_not_a_member
-
-def get_context(context):
-    context.no_cache = 1
-    context.course = frappe.form_dict["course"]
-    context.batch_code = frappe.form_dict["batch"]
-    redirect_if_not_a_member(context.course, context.batch_code)

community/www/courses/topic.html

@@ -1,108 +0,0 @@
-{% extends "templates/base.html" %}
-{% from "www/macros/livecode.html" import LiveCodeEditor with context %}
-{% block title %}{{topic.title}} ({{course.title}}){% endblock %}
-{% block head_include %}
-	<meta name="description" content="Topic {{topic.title}} of the course {{course.title}}" />
-	<meta name="keywords" content="course {{course.title}} {{topic.title}}" />
-	<style>
-	</style>
-
-<link rel="stylesheet" href="{{ livecode_url }}/static/codemirror/lib/codemirror.css">
-<link rel="stylesheet" href="/assets/css/lms.css">
-
-<script src="{{ livecode_url }}/static/codemirror/lib/codemirror.js"></script>
-<script src="{{ livecode_url }}/static/codemirror/mode/python/python.js"></script>
-<script src="{{ livecode_url }}/static/codemirror/keymap/sublime.js"></script>
-
-<script src="{{ livecode_url }}/static/codemirror/addon/edit/matchbrackets.js"></script>
-<script src="{{ livecode_url }}/static/codemirror/addon/comment/comment.js"></script>
-{% endblock %}
-
-{% block content %}
-<section class="top-section" style="padding: 1rem 0rem;">
-	<div class='container pb-5'>
-		<nav aria-label="breadcrumb">
-			<ol class="breadcrumb">
-				<li class="breadcrumb-item" aria-current="page"><a href="/courses">Courses</a></li>
-				<li class="breadcrumb-item" aria-current="page"><a href="/courses/{{course.slug}}">{{course.title}}</a></li>
-			</ol>
-		</nav>
-
-		<h1>{{ topic.title }}</h1>
-
-    {% for s in topic.get_sections() %}
-    <div class="section section-{{ s.type }}">
-      {{ render_section(s) }}
-    </div>
-    {% endfor %}
-
-	</div>
-</section>
-{% endblock %}
-
-{% macro render_section(s) %}
-  {% if s.type == "text" %}
-    {{ render_section_text(s) }}
-  {% elif s.type == "example" or s.type == "code" %}
-    {{ LiveCodeEditor(s.name, s.get_latest_code_for_user()) }}
-  {% else %}
-    <div>Unknown section type: {{s.type}}</div>
-  {% endif %}
-{% endmacro %}
-
-{% macro render_section_text(s) %}
-  <div class="row">
-    <div class="col-md-9">
-      {{ frappe.utils.md_to_html(s.contents) }}
-    </div>
-  </div>
-{% endmacro %}
-
-{%- block script %}
-  {{ super() }}
-  <script type="text/javascript" src="{{ livecode_url }}/static/livecode.js"></script>
-  <script type="text/javascript">
-    $(function() {
-      var editorLookup = {};
-
-      $(".canvas-editor").each((i, e) => {
-        var data = $(e).data();
-        var editor = new LiveCodeEditor(e, {
-          runtime: "python-canvas",
-          base_url: "{{ livecode_url }}",
-          codemirror: true,
-          userdata: data,
-          autosave: function(editor, code) {
-            // can't autosave when user is Guest
-            if (frappe.session.user == "Guest") {
-              return;
-            }
-            var data = editor.options.userdata;
-            var code = editor.codemirror.doc.getValue();
-            // console.log("autosaving...")
-            frappe.call("community.lms.api.autosave_section", {
-              section: data.section,
-              code: code
-            }).then((r) => {
-              // TODO: verify
-            })
-          }
-        })
-        editorLookup[data.section] = editor;
-      })
-
-      $(".canvas-editor .reset").each((i, e) => {
-        $(e).on("click", function(event) {
-          var data = $(this).parents(".canvas-editor").data();
-          var section = data.section;
-          frappe.call("community.lms.api.get_section", {
-            name: section
-          }).then(r => {
-            var editor = editorLookup[data.section];
-            editor.codemirror.doc.setValue(r.message.contents);
-          })
-        })
-      })
-    })
-  </script>
-{%- endblock %}

community/www/courses/topic.py

@@ -1,33 +0,0 @@
-import frappe
-
-def get_context(context):
-    context.no_cache = 1
-
-    try:
-        course_slug = frappe.form_dict['course']
-        topic_slug = frappe.form_dict['topic']
-    except KeyError:
-        context.template = 'www/404.html'
-        return
-
-    course = get_course(course_slug)
-    topic = course and course.get_topic(topic_slug)
-
-    if not topic:
-        context.template = 'www/404.html'
-        return
-
-    context.course = course
-    context.topic = topic
-    context.livecode_url = get_livecode_url()
-
-def notfound(context):
-    context.template = 'www/404.html'
-
-def get_livecode_url():
-    doc = frappe.get_doc("LMS Settings")
-    return doc.livecode_url
-
-def get_course(slug):
-    course = frappe.db.get_value('LMS Course', {"slug": slug}, ["name"], as_dict=1)
-    return course and frappe.get_doc('LMS Course', course['name'])

community/www/macros/livecode.html

@@ -24,7 +24,7 @@
 </div>
 {% endmacro %}
 
-{% macro LiveCodeEditor(name, code, is_exercise, last_submitted) %}
+{% macro LiveCodeEditor(name, code, reset_code, is_exercise=False, last_submitted=None) %}
 <div class="livecode-editor livecode-editor-inline" id="editor-{{name}}">
   <div class="row">
     <div class="col-lg-8 col-md-6">
@@ -34,10 +34,13 @@
         {% if is_exercise %}
           <button class="submit pull-right btn-primary">Submit</button>
           {% if last_submitted %}
-          <span class="pull-right" style="padding-right: 10px;">Submitted <span class="human-time" data-timestamp="{{last_submitted}}">on {{last_submitted}}</span></span>
+          <span class="pull-right" style="padding-right: 10px;"><span class="human-time" data-timestamp="{{last_submitted}}"></span></span>
           {% endif %}
         {% endif %}
       </div>
+      <div style="display: none">
+        <pre class="reset-code">{{reset_code}}</pre>
+      </div>
     </div>
   </div>
   <div class="code-editor">
@@ -59,20 +62,67 @@
 
 
 {% macro LiveCodeEditorJS(name, code) %}
+
+<script type="text/javascript" src="/assets/frappe/node_modules/moment/min/moment-with-locales.min.js"></script>
+<script type="text/javascript" src="/assets/frappe/node_modules/moment-timezone/builds/moment-timezone-with-data.min.js"></script>
+<script type="text/javascript" src="/assets/frappe/js/frappe/utils/datetime.js"></script>
+
+<script type="text/javascript">
+// comment_when is failing because of this
+if (!frappe.sys_defaults) {
+  frappe.sys_defaults = {}
+}
+</script>
+
 <script type="text/javascript" src="{{ livecode_url }}/static/livecode.js"></script>
 <script type="text/javascript" src="/assets/community/js/livecode-canvas.js"></script>
 
 <script type="text/javascript">
   var livecodeEditors = [];
+  var livecodeEditorsMap = {};
 
   $(function() {
     $(".livecode-editor").each((i, e) => {
+      var name = e.id.replace("editor-", "");
       var editor = new LiveCodeEditor(e, {
         base_url: "{{ livecode_url }}",
         ...getLiveCodeOptions()
       })
       livecodeEditors.push(editor);
-    })
+      livecodeEditorsMap[e.id] = editor;
-  })
+
+      $(e).find(".reset").on('click', function() {
+        let code = $(e).find(".reset-code").html();
+        editor.codemirror.doc.setValue(code);
+      });
+
+      $(e).find(".submit").on('click', function() {
+        let code = editor.codemirror.doc.getValue();
+        console.log("submit", name, code);
+        frappe.call("community.lms.api.submit_solution", {
+          "exercise": name,
+          "code": code
+        }).then(r => {
+          if (r.message.name) {
+            frappe.msgprint("Submitted successfully!");
+
+            let d = r.message.creation;
+            $(e).find(".human-time").html(__("Submitted {0}", [comment_when(d)]));
+          }
+        });
+      });
+
+    });
+  });
+
+  function updateSubmitTimes() {
+    $(".human-time").each(function(i, e) {
+      var d = $(e).data().timestamp;
+      $(e).html(__("Submitted {0}", [comment_when(d)]));
+    });
+  }
+
+  updateSubmitTimes();
 </script>
+
 {% endmacro %}

community/www/macros/sidebar.html

@@ -1,12 +1,14 @@
-{% macro Sidebar(course, batch_code) %}
+{% macro Sidebar(course, batch, is_mentor=False) %}
 <div class="sidebar-batch">
-    <a href=""><i class="fas fa-bars fa-lg"></i></a>
+    <a href=""><i class="fa fa-bars fa-lg"></i></a>
     <br>
-    <a href="/courses/{{course}}/{{batch_code}}/learn"><i class="fas fa-book fa-lg"></i></a>
+    <a href="/courses/{{course.name}}/{{batch.name}}/learn"><i class="fa fa-book fa-lg"></i></a>
-    <a href="/courses/{{course}}/{{batch_code}}/schedule"><i class="fas fa-calendar-alt fa-lg"></i></a>
+    <a href="/courses/{{course.name}}/{{batch.name}}/schedule"><i class="fa fa-calendar fa-lg"></i></a>
-    <a href="/courses/{{course}}/{{batch_code}}/members"><i class="fas fa-users fa-lg"></i></a>
+    <a href="/courses/{{course.name}}/{{batch.name}}/members"><i class="fa fa-users fa-lg"></i></a>
-    <a href="/courses/{{course}}/{{batch_code}}/discuss"><i class="fas fa-comments fa-lg"></i></a>
+    <a href="/courses/{{course.name}}/{{batch.name}}/discuss"><i class="fa fa-comments fa-lg"></i></a>
-    <a href="/courses/{{course}}/{{batch_code}}/about"><i class="fas fa-info-circle fa-lg"></i></a>
+    <a href="/courses/{{course.name}}/{{batch.name}}/about"><i class="fa fa-info-circle fa-lg"></i></a>
-    <!-- <a href="#contact"><i class="fas fa-home fa-lg"></i></a> -->
+    {% if batch.is_member(frappe.session.user, member_type="Mentor") %}
+      <a href="/courses/{{course.name}}/{{batch.name}}/progress"><i class="fa fa-flag-checkered fa-lg"></i></a>
+    {% endif %}
 </div>
 {% endmacro %}