-
Notifications
You must be signed in to change notification settings - Fork 305
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
Meta enrolment in csv course upload #815
Meta enrolment in csv course upload #815
Conversation
ilyatregubov
commented
Nov 17, 2023
✅ Deploy Preview for moodledevdocs ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
⚡️ Lighthouse report for the deploy preview of this PR
|
a355b5f
to
2df22c3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @ilyatregubov!
Thanks a lot for preparing a patch for this. As you'll see, I added a few minor proposals to improve it :-)
MDL-73852 changes
1935ad9
to
3f02243
Compare
All issues have been resolved. Just need someone to approve |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @ilyatregubov!
Thanks for reviewing the patch. I'm happy to approve and merge it. Well done!