-
Notifications
You must be signed in to change notification settings - Fork 13
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
Pending pull request #15
Comments
Hi @rschrenk , I am sorry that I did not have the time to review your PR and finally merge it last year. We went through a hard phase last year where we had to pause our Moodle development and only did the most necessary things like upgrading the plugins to Moodle 3.10. I can assure you that your PR #11 is not lost and still open and it still can be merged into the plugin. This plugin will also soon get an additional maintainer and we will see when we can work through your useful addition in #11. Cheers, |
Hi @abias we will do our Moodle updates in mid July. Is there a chance to get this PR done till then? Regards |
Hi @abias Is there chance, that you pull my changes into the plugin now? You also developed new features, so I don't think I should work on my fork, but my changes may be of interest for all users of this plugin. Best regards |
I would also like to express my support for this feature. If I can help, please feel free to let me know! |
Dear Alexander,
we had contact regarding the merger of my plugins availability_globalrole and availability_coursecatrole into availability_role. I made all required changes and set up a pull request, but it has not been accepted.
In the meanwhile it seems you have been working on the plugin, so my functionality is lost when updating.
Could you please integrate my changes?
#11
Best wishes
Robert
The text was updated successfully, but these errors were encountered: