-
Notifications
You must be signed in to change notification settings - Fork 6
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
Proposal: Add feature for resetting user data #99
Comments
Hi Andreas, thanks a lot for your feedback and your feature proposal! User data is stored outside of mod_learningmap as it depends on course completion data from moodle core. For providing such a feature, it had to override course module completion of the modules used in the learning map as report_progress does. I'll have to think a bit about that, I'm not really sure if this is what you want to have. Best regards |
Dear Stefan, thanks for your immediate response. I noticed that you store user data in "[mdl_]course_modules_completion". OK, you think about if if you have enough time. ;-) Best regards |
Hi Andreas, this is not user data created by mod_learningmap itself - Moodle core uses this table to store course module completion (which is used by mod_learningmap). If you look at the source code of mod_learningmap, you won't find any direct database access to this table. Best regards |
Hi Stefan, thanks for clarifying. I should have been more precise: "[mdl_]course_modules_completion" is used to display the learning progress on the map. Best regards |
I suggest to convert this into a feature request like this:
Note: Besides the fact that of course the completion data of the single activities are being cleared (including automatic completion data which could be a bit tricky to solve) this would also possibly influence other learningmap instances linking to the same activities. |
Dear Stefan,
thanks for your great plugin. It would be great if user data could be deleted so that a learning map can be discovered from scratch.
This would be especially useful for testing purposes.
Best regards
Andreas
The text was updated successfully, but these errors were encountered: