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

Mark custom certificate as viewed when emailing the certificate #645

Open
izendegi opened this issue Oct 2, 2024 · 5 comments
Open

Mark custom certificate as viewed when emailing the certificate #645

izendegi opened this issue Oct 2, 2024 · 5 comments

Comments

@izendegi
Copy link

izendegi commented Oct 2, 2024

The automatic completion settings for this activity only give the option to be based on the student having viewed the activity, but we've seen that many students don't view the activity because they also received the certificate by email.

Because of that, many students appear to haven't completed the activity in the activity completion reports even if they've already received the certificate.

Having that into account, it would be great if there were another completion requirement option based on having received the certificate.

@mdjnelson
Copy link
Owner

mdjnelson commented Oct 2, 2024

The plugin contains no logic involving the conditions required to access an activity. This is a feature available to all activities via the Moodle API. I guess one solution to this would be to trigger a completion event when it gets emailed, though I am not too sure if that would work. Do you have the resources to confirm this?

@izendegi
Copy link
Author

izendegi commented Oct 2, 2024

Hi, @mdjnelson

I'm sorry, I think I haven't explained well what I mean :-/

My point wasn't about access restriction conditions but activity completion conditions, specifically those that can be achieved automatically; currently the custom certificate activity gives only one option of this kind of requirements ("View the activity"), which I believe a standard one:
irudia

Other activities gives more requirements to choose from depending on each activities' logic; some of them also give a "Receive a grade" requirement and mod_subcourse for example gives also a "Student must complete the referenced course to complete this activity" requirement
irudia

So, what I want to propose is to add a new one this kind of requirements to the mod_customcert that depends on the student receiving the certificate in addition to the current "View the activity" requirement (or even replacing it), because I believe that this requirement is the most accurate of completing the activity.

Is this way, those places which show the completion status of the activities (like the activity completion report or the completion progress block) would show a more accurate status of the completion of this activity.

This change would be also very helpful if you want to add completing the certificate activity to the course completion criteria or as a access restriction for another activity (currently only those students who have accessed the activity are meeting the criteria, not those who received the certificate).

@mdjnelson
Copy link
Owner

Ah, I see.

We have https://github.com/mdjnelson/moodle-mod_customcert/blob/MOODLE_404_STABLE/view.php#L196 which marks the certificate as viewed which ensures the completion status is set. Perhaps this needs to be done when it is emailed as well. If so, this could probably be defined as a bug.

Thoughts?

@izendegi
Copy link
Author

izendegi commented Oct 2, 2024

Yes, I think that marking that requirement as completed also when the certificate is emailed to the student would work.

Using that approach (changing the current requirement to include all the students that have received the certificate), I would suggest to change also the requirement's name to something more accurate ("Receive the certificate" or something similar).

@mdjnelson mdjnelson changed the title Improvement proposal: add a completion requirement option based on having received the certificate Mark custom certificate as viewed when emailing the certificate Oct 2, 2024
@mdjnelson
Copy link
Owner

mdjnelson commented Oct 2, 2024

I would have to add more logic to the plugin in order to have another field, which means more things I need to maintain. That current condition in the certificate is there due to Moodle having it as a default for activities that don't define anything. The sub course plugin defines extra criteria, see https://github.com/catalyst/moodle-mod_subcourse/blob/MOODLE_404_STABLE/mod_form.php#L173. I renamed this issue to reflect what I will do when it is addressed.

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

No branches or pull requests

2 participants