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 option to not track completion of reengagement #45

Open
austin-powell opened this issue Sep 13, 2018 · 8 comments
Open

Add option to not track completion of reengagement #45

austin-powell opened this issue Sep 13, 2018 · 8 comments

Comments

@austin-powell
Copy link

Based on my understanding of how this code works, I'm not sure that this is possible without a complete rewrite but I'll ask anyway.

Is there a reason why reengagements are forced to have activity completion turned on for them with "Show activity as complete when conditions are met"?

This causes all reengagements to get counted by Moodle's default get_course_progress_percentage function that is used (among other places) in the updated course overview block for Moodle 3.3 and on. The reason why this is an issue is that we for example have an reengagement set to occur 5 minutes after enrollment that sends a welcome email to these students. But because this reengagement then gets counted as "complete" users who have never logged in still show 4% complete on anything using core Moodle's progress functions including our custom progress reports.

Is it possible to change this or am I correct in my understanding of the plugin and its coding?

@peterbulmer
Copy link
Member

Hi Austin,
It's been a while since I looked at the relevant code, but my recollection is that it wouldn't require a complete rewrite, and 'show activity complete' would likely have been set under the assumption that it was harmless.
If anyone wants to look into this issue don't be discouraged - it might be a minor-effort thing.

@jeremy-schweitzer
Copy link

I agree that it would be helpful to have an option to set completion as none/not tracked.

@danmarsden
Copy link
Member

Feel free to help us work on this! - pull requests and or funding are always welcome! :-)

@ccomincini
Copy link

Good morning, I believe that the option to avoid tracking the completion of the activity would be necessary, especially considering that many users complete the courses without completing the reengagement and that moodle dashboard does not report 100% completion on the course because of the failed completion of the reengagement activity.
This is a problem and causes many support tickets are opened by users who know they have completed the course and want to see 100% completion in the dashboard.

@Lukas-Sander
Copy link

After some digging and searching, I found the code lines that need to be commented out for you to be able to select a different completion tracking (e.g. not tracking).
You need to comment out the following lines in mod_form.php:

184-187: if ($mform->elementExists('completion')) { [...]
266: $toform->completion = COMPLETION_TRACKING_AUTOMATIC;
288: $fromform->completion = COMPLETION_TRACKING_AUTOMATIC;

@Urpokarhu1
Copy link

Any progress on this fix?

@danmarsden
Copy link
Member

@Urpokarhu1, feature development on our plugins typically requires someone to fund the time for our development team to implement the improvements. Feel free to reach out privately if this is something you are interested in or if you have internal development capabilities, pull requests are always very welcome!

@abautu
Copy link

abautu commented Aug 26, 2024

I have faced this problem also (i.e. user completes all course modules but the course progress is missing completion the Reengagement module, and they will stay like that for the duration setting of the reengagement module).

This happens even if the "Suppress notification if target activity complete" is enabled.

I solved this issue with this #169, that makes reengagement to become completed if the user already completed the "suppress" activity.

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

No branches or pull requests

8 participants