-
Notifications
You must be signed in to change notification settings - Fork 161
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
Moodle 4.1 - PHP 8.1 | Work #241
Comments
Hi Gemma, that's great new, I will review the changes once submitted |
Wow, this is great. Question, because this is not clear from what won't be done "Maintaining the further code.". Will it be compatible with 4.3 too? |
@fabitas what we mean is that we won't be actively maintaining the code or adopting the plugin. So to answer your questions: if it works with Moodle 4.3 need to be tested by the users and issues should be reported in this github. We are right now testing 4.1 version |
@jleyva we have finished the work for Moodle 4.1 , php 8.1 version. We are currently testing the plugin with a small community group. We are expecting to hand over the code to you at the end of February so you can do a follow up check and release it to Moodle.org , would that work for you? |
@jleyva I just send a pull request. Even though we tested it, I think a code review would be a good idea before you merge it and release it to the community if there are any questions please let me know. |
Thanks, I'm reviewing this but it is a huge patch! So it will take some time |
Are your changes incorporated in the new 4.1.0 release (https://github.com/jleyva/moodle-block_configurablereports/releases/tag/4.1.0)? |
@jleyva We just want to inform you that we will be working on a M4.1 / PHP 8.1 release on the Configurable Reports block. This work will be done by Luuk Verhoeven.
The forcus will be:
What won't be done is:
We expect to be able to test the new version around January 12th 2023, when we feel comfortable enough we will create a pull request and hopefull you are willing to check (if necessary) and release this new branche/request to Moodle.org/plugins
In case you have any questions or feel like certain Issues mentioned in this tracker can be part of the "Focus will be" section, It would be great if you can add it this issue.
Greetings
Gemma Lesterhuis - Lesterhuis Training & Consultancy
The text was updated successfully, but these errors were encountered: