Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Class flow #580

Merged
merged 8 commits into from
Aug 10, 2023
Merged

feat: Class flow #580

merged 8 commits into from
Aug 10, 2023

Conversation

pateljannat
Copy link
Collaborator

@pateljannat pateljannat commented Aug 7, 2023

  • For a class, moderators can now define a schedule for each lesson of the courses included in the class.
  • From the LMS Class DocType, after adding courses to a class, they can add the schedule.
  • All they have to do is click on the Fetch Lessons button and this will automatically fetch all the lessons in the order of courses.
  • You can then add a Date, Start Time, and End Time to each lesson. The system will validate if the date and time fall within the range of the date and time of the class.
Screenshot 2023-08-10 at 12 13 47 PM
  • Once the schedule has been set, a student of the class will be able to see the schedule on the class page. They can click on the lesson and see the content of the lesson there.
  • The schedule page also shows if the student has completed the lesson.
Screenshot 2023-08-10 at 12 28 48 PM

@cypress
Copy link

cypress bot commented Aug 7, 2023

Passing run #170 ↗︎

0 1 0 0 Flakiness 0

Details:

Merge 03620be into 66dbe68...
Project: lms Commit: 9583f5269f ℹ️
Status: Passed Duration: 00:58 💡
Started: Aug 10, 2023 7:34 AM Ended: Aug 10, 2023 7:35 AM

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

@pateljannat pateljannat changed the title Class flow feat: Class flow Aug 9, 2023
@pateljannat pateljannat marked this pull request as ready for review August 10, 2023 07:31
@pateljannat pateljannat merged commit 9cdc8a5 into frappe:main Aug 10, 2023
3 checks passed
@pateljannat pateljannat mentioned this pull request Aug 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant