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

Add: Option to unenrol from courses #336

Open
wants to merge 1 commit into
base: development
Choose a base branch
from

Conversation

abhijeetviswa
Copy link
Member

@abhijeetviswa abhijeetviswa commented Sep 1, 2021

This commit introduces a bunch of code to add the ability to unenrol
students from courses. The method is hacky, involving creating an HTTP
session using a private token, followed by emulating a HTML form submit
to do the unenrolment. Unfortunately, Moodle doesn't have a webservice
to allow for unenrolment. :(

While the core functionality has been implemented, this PR isn't complete yet. There's a bunch of UI/UX things that need to be cleaned up:

  • Course page still open even after unenrolment.
  • Local course list does not refresh after unenrolment.

This commit introduces a bunch of code to add the ability to unenrol
students from courses. The method is hacky, involving creating an HTTP
session using a private token, followed by emulating a HTML form submit
to do the unenrolment. Unfortunately, Moodle doesn't have a webservice
to allow for unenrolment. :(
@abhijeetviswa abhijeetviswa self-assigned this Sep 1, 2021
@SKrPl
Copy link
Member

SKrPl commented Sep 28, 2021

@abhijeetviswa what about creating a PR on the moodle for an unenroll API endpoint? Are there any students who use PHP?

@abhijeetviswa
Copy link
Member Author

abhijeetviswa commented Sep 28, 2021

@SKrPl Tried that 🥲
https://tracker.moodle.org/browse/MDL-64255

Didn't create the patches they asked for. Hopefully someone will get around to it someday.

sakets3010 added a commit to sakets3010/CMS-Android that referenced this pull request Jan 9, 2022
…urses fragment after unenrolment

2 fixes were carried out on pull request crux-bphc#336 , namely :-
1)Course page still opens even after unenrolment.
2)Local course list does not refresh after unenrolment.
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.

None yet

2 participants