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

fix: Remove HTML from settings descriptions #28946

Merged
merged 41 commits into from
Jun 13, 2023

Conversation

rique223
Copy link
Contributor

@rique223 rique223 commented Apr 17, 2023

Proposed changes (including videos or screenshots)

Setting descriptions stopped accepting HTML and because of that several setting had incorrect descriptions.

HTML tags/formatting was replaced by markdown.

Issue(s)

fix #28960

Steps to test or reproduce

Further comments

WM-1

@codecov
Copy link

codecov bot commented Apr 18, 2023

Codecov Report

Merging #28946 (4ae8235) into develop (7cbf4d4) will increase coverage by 0.07%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop   #28946      +/-   ##
===========================================
+ Coverage    46.68%   46.75%   +0.07%     
===========================================
  Files          698      699       +1     
  Lines        13071    13087      +16     
  Branches      2226     2228       +2     
===========================================
+ Hits          6102     6119      +17     
+ Misses        6652     6647       -5     
- Partials       317      321       +4     
Flag Coverage Δ
e2e 46.72% <ø> (+0.08%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

@hugocostadev hugocostadev marked this pull request as ready for review April 26, 2023 12:00
KevLehman
KevLehman previously approved these changes May 30, 2023
Copy link
Contributor

@KevLehman KevLehman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Omni files good

KevLehman
KevLehman previously approved these changes May 30, 2023
debdutdeb
debdutdeb previously approved these changes May 30, 2023
@hugocostadev hugocostadev dismissed stale reviews from debdutdeb and KevLehman via 4ae8235 June 12, 2023 12:50
@casalsgh casalsgh requested review from a team and removed request for a team June 12, 2023 16:55
@hugocostadev hugocostadev dismissed ggazzo’s stale review June 13, 2023 12:32

Not a bug, it's a improvement that will be done later

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

HTML tags are visible inside most of the Settings Option in the Administration page.
6 participants