Skip to content

Conversation

@b0ink
Copy link

@b0ink b0ink commented May 7, 2025

Description

This feature will allow staff to enable the "Tutorial self enrolment" feature for a designated task. Staff will select which tutorial stream(s) to fetch the tutorials from, and students will be prompted with the list of tutorials they must choose from to complete the task.

How has this been tested?

Ensure you are also using the Front End branch:
thoth-tech/doubtfire-web#306

  1. Login as aadmin
  2. Select any unit and any task to edit
  3. Scroll to the "Tutorial enrolment form" and enable it
  4. Select a tutorial stream that has a atleast a few tutorials
  5. If there are no tutorial streams to choose from or the stream has no tutorials, manually add them in the in the tutorials tab.
  6. Save the changes to the task after enabling the self enrolment form.
  7. Now login as student_1, and navigate to the same unit and task you enabled the form for.
  8. Click on "Begin enrolment", and confirm the list of tutorials show up. Do note that some tutorials may not show up for a student based on their campus. Eg. if a student is based on campus X, but the tutorial is only available for campus Y, the student will not see it.
  9. Select a tutorial and click enrol.
  10. If it says you have already been enrolled, you can navigate to the Tutorial List and withdraw yourself from the unit, then go back to the task to begin tutorial enrolment, and select a tutorial

Type of change

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

@b0ink b0ink marked this pull request as ready for review May 14, 2025 06:40
Copy link

@aditya993388 aditya993388 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @b0ink , Please disregard the approval, I have requested changes. Everything in your PR looked fine, but when I tried running the frontend and backend branches locally, I repeatedly encountered database migration errors. Is there any additional setup required to run this correctly? If so, could you please document it? I've attached the screenshots below.

Frontend:
image

Backend:
image
image

amriith

This comment was marked as off-topic.

Copy link

@aditya993388 aditya993388 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @b0ink , Everything in your PR looked fine, but when I tried running the frontend and backend branches locally, I repeatedly encountered database migration errors. Is there any additional setup required to run this correctly? If so, could you please document it? I've attached the screenshots below.

Frontend:
image

Backend:
image
image

Copy link

@Duy-Nguyen1104 Duy-Nguyen1104 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi. I was able to pull the branch 8.0.x and tested out the self enrolment feature of this pull requests. Initially, when switching branches and re-run dev container, the database wasn't auto filled with test users and courses, and I had to manually run db:init and db:populate in the backend (don't know if this problem happened only on my machine or not).

I was able to enable self tutorial enrolment for tasks and choosing between tutorial streams.
image
When logged in as student, I noticed that student from the selected streams are auto enrolled in the tutorial enrolment, and I can withdraw from or enrol to which tutorial I choose.
image

There is just a minor code redundancy in your files changed. Overall, great job!

expose :scorm_attempt_limit
expose :tutorial_self_enrolment_enabled
expose :tutorial_self_enrolment_stream_id
expose :scorm_attempt_limit

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This line I noticed to be redundant as it is already written on line 50

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.

4 participants