-
Notifications
You must be signed in to change notification settings - Fork 451
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
Redesigned Workflow - Discussions #10762
Comments
One slight issue @jardakotesovec with the title of the comment with characters that are not alphanumeric. This didn't happen on 3.4 version. Don't know what the expectations are for types of acceptable characters. Here are some happy path tests that I conducted. It seems to be working as expected except for that minor formatting bug. Test case categories
|
@Tribunal33 That problematic rendering of title is now addressed. Reason being, that with 3.5 the modal titles are rendered via vue.js, which does the xss protection. So had to remove explicit html escaping which was in place before. |
Fixed. This ticket now passes QA. I updated a test to check for the encoding issue (non-alphanumeric characters). Test case categories
|
Discussions are taken from 3.4, without changes. This ticket is just for integration testing. Basically ensure that each of these discussion grid are correctly linked to correct stage.
Easiest to test this is probably just to use the feature flag to switch between legacy and new workflow to verify if message is added in one stage it will appear in that same stage both in the legacy workflow and the new one.
The text was updated successfully, but these errors were encountered: