bcgov / moodle

Repo to store codebase for the Moodle app
Apache License 2.0
1 stars 1 forks source link

Install Moodle Plugin - Cerificate #90

Open warrenchristian1telus opened 2 years ago

warrenchristian1telus commented 2 years ago

Install plugin to dev, then test for review by Shannon & Greg.

@allanhaggett Can you please confirm which one we want? It seems they are both current, but only the newer one is receiving feature updates, while the older version will only receive bug fixes.

https://moodle.org/plugins/mod_certificate Or (newer) https://moodle.org/plugins/mod_customcert

The newer one can be customized in the browser, while the older one requires PHP customization. This could affect how we manage / migrate changes, as the older PHP customization should integrate well with GitHub workflows, while the newer is most likely stored in the database, so would require alternate backup / migration techniques. After some research, it seems the new version includes backup / restore functionality, which appears to work similar to the course backups.

Apparently, both can be installed at once without issue, if we want to test and validate both options in the same environment.

allanhaggett commented 2 years ago

The older one being better for GH integration is a fantastic point.

Install them both. We'll evaluate and if the older one doesn't fit, then we'll just remove it.

Thanks!

warrenchristian1telus commented 2 years ago

This issue has been resolved by @jizhaogit. Merged to main branch and deployed to dev namespace for review.

warrenchristian1telus commented 2 years ago

https://github.com/bcgov/moodle/pull/103

warrenchristian1telus commented 2 years ago

These plugins have been implemented in dev. Awaiting testing / review before migrating (one of them) to test.

allanhaggett commented 2 years ago

Acknowledged! Thank you! I have brought this event to our course developers who are now in the process of reviewing and testing these. I'll be meeting with them again early next week to deep-dive and come back with feedback on which certificates plugin we want to use.

allanhaggett commented 2 years ago

We still need to make a final recommendation here. Notwithstanding any sustainability concerns from Warren, after conferring with the course dev team, we are hoping to stick with the newer of the two choices, though we don't have a very strong opinion as both options fulfill the requirements.

Not going to close this for now, but it's not a real priority IMHO so we should put it on the back burner until enrolment ELM bridge is completed,