Reply to - topic bug

Contribute Bug

AI-generated summary

A bug has been reported in the latest version of Discourse (3.4.0.beta4-dev) on topic bug, where users are experiencing issues with replying to topics. twofoursixeight initially reported the bug and provided screenshots, which was then reproduced by CocoQuark on try.discourse.org. Arkshine identified the issue as a side-effect of a specific commit and noted that the translatedLabel function now requires explicit declaration of HTML safety. A fix is being worked on, as confirmed by Moin and sam, with a pull request (#30962) already in progress.

1

Forum bug happening on the latest version of Discourse installed in forum.creative.gimkit.com

The image displays a webpage containing HTML code snippets for "Your Question" and "Your Reply" topics in a text area where users can enter forum interactions, including bug reports relating to a Gimkit Creative service. (Captioned by AI)
The image displays a webpage containing HTML code snippets for "Your Question" and "Your Reply" topics in a text area where users can enter forum interactions, including bug reports relating to a Gimkit Creative service. (Captioned by AI)503×717 63.1 KB

The version of the forum running is 3.4.0.beta4-dev right now.
Also reproduced this bug on try.discourse.org :
The image shows a user editing HTML code for a Discord topic within the Discord user interface, discussing how to reply to topics and how to hide or unhide a new reply message. (Captioned by AI)
The image shows a user editing HTML code for a Discord topic within the Discord user interface, discussing how to reply to topics and how to hide or unhide a new reply message. (Captioned by AI)1920×988 190 KB

4 Likes
2

I can repro on try.discourse as well :+1:

1 Like
3

It’s a side-effect of this commit: DEV: Only mark translations as htmlSafe for DButton (#30958) · discourse/discourse@2b63830 · GitHub

It seems you now must be explicit that a label is HTML-safe when using translatedLabel.
In the dialog context, it misses one htmlSafe here. However, I don’t know if it impacts others’ codes yet.

https://github.com/discourse/discourse/blob/main/app/assets/javascripts/dialog-holder/addon/components/dialog-holder.gjs#L61

2 Likes
4

I think someone is already working on a fix.
https://github.com/discourse/discourse/pull/30962

5 Likes
5

We are actively working on a fix, should land one shortly

10 Likes
6

This topic was automatically closed after 2 days. New replies are no longer allowed.