Skip to content
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

Defect 828690: Behat: mod/studentquiz (tt) - Test delete comment feat… #508

Merged
merged 1 commit into from
Oct 11, 2024

Conversation

BruceGoodGuy
Copy link
Contributor

@BruceGoodGuy BruceGoodGuy commented Oct 9, 2024

Hi @AnupamaSarjoshi,
CC @timhunt,

I have completed the development for this ticket. The Behat failure was caused by an issue where, (maybe) due to network instability, the Tiny editor wasn't fully loaded when we tried to set the event listener, resulting in the event not being properly attached. To fix this, I’ve updated the initBindEditor method to ensure that the Tiny editor is fully loaded before proceeding with further actions.

Additionally, I’ve disabled the "Add reply" button initially, as the reply content is always empty at the start, to keep it consistent with the behavior of the "Add comment" button.

Please note, the JavaScript code in this file isn't written in a modern style, so I have not converted it to ES6. I believe this can be resolved in a separate ticket later on.

Could you please review the changes?

Thanks.

…ure. and Test report comment feature. #828690
@AnupamaSarjoshi AnupamaSarjoshi merged commit 83c1a4f into studentquiz:main Oct 11, 2024
3 checks passed
@AnupamaSarjoshi
Copy link
Contributor

Thanks so much @BruceGoodGuy. This looks good and is merged :)

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