Closed HuongNV13 closed 4 years ago
Tl;dr; It is spot on. Thanks.
There was one thing I was worried about:
I tried to examples in core Moodle code, but just ended up getting totally confused:
My guess is that you should do format_text(..., FORMAT_HTML).
However, looking through all the examples in the OU code, we just use the value directly. So I guess your code is consistent with all our other examples, and we should stick with that. Therefore, I will merge this pull request.
Hi @timhunt ,
Please help me to review it
The code was tested done by our tester
Thanks,