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

Form for each student #257

Open
wants to merge 18 commits into
base: master
Choose a base branch
from

Conversation

danielneis
Copy link

Hello,

This adds an option on mod_form to generate one answer form per student, pre filling their idnumber, firstname and lastname,
The generate file is a unique PDF for all students.
I've not indented the main part of the code intentionally to easy the review process, but once you approve it I'll make that change.

Best,
Daniel

@univietw univietw changed the base branch from MOODLE_403_STABLE to master May 15, 2024 10:37
@univietw
Copy link
Contributor

Hi Daniel,

thank you for your pull request.

We are generally very interested in your changes. There are just some caveats whe have to solve before fixing this issue:

  1. This feature would be for the next version (offlinequiz 4.4) first. I changed the branch to master for this. It would be nice if you could rebase your changes to that branch too.
  2. We would like to have an admin setting so the admins can disable this feature for new offline quizzes.
  3. One important question is the allocation: Which user gets which group. There is no point in printing 6 pdfs for each user, if there are 6 different groups. Maybe you could use the participants lists for that? It's definitely something to be discussed.
  4. If the setting is enabled there should also be the empty one be included. Teachers should be able to print this one out for emergencies.

That's the immediate feedback. I will come back later with some more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants