-
Notifications
You must be signed in to change notification settings - Fork 24
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
Start integrating the feature into Moodle core #117
Comments
hi @juho-jaakkola I'll add some thoughts to that tracker |
Hi @juho-jaakkola, check out how I implemented routing in my own plugin. I talk about it in #116. I do agree that such a feature should be in core instead though. |
Getting slightly off-topic, but I'll write this down somewhere before I forget it: Besides front controller, I've also been looking into making Moodle installable as a composer dependency. I wonder if the front controller could be implemented (at least as the first step) into the composer project base instead of Moodle core. Take a look for example at these two examples, where not only the modules/plugins/libraries, but also the version of the framework itself is managed through a single This approach would allow:
|
I recently made a feature request about a front controller for Moodle: https://tracker.moodle.org/browse/MDL-60140
Now I came across your plugin, and realized you've already started implementing that feature!
I bet you have plenty of insight on the matter, so it would be great if you could provide some kind of list or roadmap of needed changes into the feature request.
It would great if all the future work would go directly into Moodle core as an officially supported feature.
The text was updated successfully, but these errors were encountered: