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

MBS-9026 implement reset userdata #548

Open
wants to merge 2 commits into
base: stable
Choose a base branch
from

Conversation

pmaneggia
Copy link

Also delete entries in hvp_content_user_data when deleting instance

@pmaneggia
Copy link
Author

This covers Issue #150
Doing this work I noticed that when deleting an instance of the activity (callback hvp_delete_instance in lib.php) does not clear the table hvp_content_user_data accordingly, so I added this as well.

@pmaneggia pmaneggia force-pushed the add-reset-support-in-moodle-course branch from b911e8d to 2773137 Compare May 7, 2024 15:55
@pmaneggia
Copy link
Author

I just force pushed a small improvement to the commit assuming that no one is actually working with this. Apologies if this was not the case.

Also delete entries in hvp_content_user_data when deleting instance
@pmaneggia pmaneggia force-pushed the add-reset-support-in-moodle-course branch from 2773137 to 45efaaa Compare May 7, 2024 17:20
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.

1 participant