Skip to content

Conversation

dvdcastro
Copy link

No description provided.

@dmitriim
Copy link
Member

dmitriim commented Sep 3, 2024

Thanks @dvdcastro for proposed patch.
Can you please add unit tests to support your change? Otherwise LGTM.

@dmitriim
Copy link
Member

Hi @dvdcastro
Can you please rebase your change on top of MOODLE_404_STABLE branch. Once merged changes can be backported to earlier versions.

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.

2 participants