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

Add in video quiz and topic contents for a lesson #764

Closed
wants to merge 20 commits into from

Conversation

adityacp
Copy link
Contributor

This PR closes #739

  • Allow moderator to add quizzes and topic for the lesson video.
  • Add a JS player which supports YouTube, Vimeo and general video URL.
  • Automatically show the quiz and topic description based on the selected time in the video.
  • Show lesson quiz statistics.
  • Add a proper markdown editor for module and lesson description.
  • Following addons are required as mentioned in the issue Few addons for the in video table of contents #763 which will be done in the upcoming PR.

- Disable the question type in video question form
- Change urls to add graded quiz, exercise, poll type question
- Save the topic and question in table of contents
- Rename lesson attribute in TableOfContents model
- Remove all alerts and add toast messages
- Accept user submissions for the lesson quiz and evaluate
- Initial lesson statistics
- Show lesson topic description as per the video time
- Fix description preview for module and lesson
- Order table of contents by time
@adityacp adityacp changed the title Add in video quiz and topic contents Add in video quiz and topic contents for a lesson Sep 15, 2020
@codecov-commenter
Copy link

codecov-commenter commented Sep 21, 2020

Codecov Report

Merging #764 into master will decrease coverage by 0.09%.
The diff coverage is 78.26%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #764      +/-   ##
==========================================
- Coverage   81.57%   81.47%   -0.10%     
==========================================
  Files          43       43              
  Lines        6036     6474     +438     
  Branches      921     1008      +87     
==========================================
+ Hits         4924     5275     +351     
- Misses        789      842      +53     
- Partials      323      357      +34     
Impacted Files Coverage Δ
yaksh/urls.py 100.00% <ø> (ø)
yaksh/templatetags/custom_filters.py 88.11% <66.66%> (-3.00%) ⬇️
yaksh/models.py 87.76% <73.07%> (-1.15%) ⬇️
yaksh/views.py 74.03% <79.76%> (+1.03%) ⬆️
yaksh/forms.py 84.93% <84.84%> (-0.08%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update db5ea3d...4ab3353. Read the comment docs.

- Add max_upload_size in settings for video file
- Change error messages in forms
- Set the video_file field max_length
- Fix add_lesson template for uploading video file
@adityacp
Copy link
Contributor Author

Opened another PR #774. Closing this PR as it is not required.

@adityacp adityacp closed this Sep 30, 2020
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.

Addons with lessons
2 participants