search

wooclap / moodle-mod_wooclap

Moodle plugin for Wooclap (https://www.wooclap.com/).
2 stars 5 forks source link

[wooclap/moodle-mod_wooclap#11] upgrade fix #12

Closed cperves closed 2 years ago

cperves commented 2 years ago

Hello, since I need to upgrade wooclap on my multiple moodle instances, sharing same code but not always using wooclap plugin I try to make a patch to enable upgrade when accesskey is empty This will avoid moodle upgrade to execute all the remote wooclap v3 upgrade webservices I add a cli script that enable an admin to execute the v3 remote upgrade instructions later May be adding a possibility to detect if remote wooclap v3 upgrade was already made should be a great improvement

Feel free to review my code, it is just a suggestion

Regards Céline

mehira-wooclap commented 2 years ago

Thank you @cperves. We will deploy your fix in the next release. hopefully very soon.

dfdeagle47 commented 2 years ago

@cperves Included in the new release here and here. Thank you for the PR!

There's now a CLI script to trigger the migration if the keys weren't present during the upgrade.