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.

Sign up for GitHub

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

Jump to bottom

[Admin] escape values on UI #13129

Merged
merged 3 commits into from Sep 13, 2022
Merged

[Admin] escape values on UI #13129

merged 3 commits into from Sep 13, 2022

Conversation

dvesh3
Copy link
Contributor

@dvesh3 dvesh3 commented Sep 12, 2022

Changes in this pull request

Resolves #

Additional info

@dvesh3 dvesh3 added the Bug label Sep 12, 2022
@dvesh3 dvesh3 added this to the 10.5.6 milestone Sep 12, 2022
@github-actions
Copy link

github-actions bot commented Sep 12, 2022 •
edited by aryaantony92

Review Checklist

  • Target branch (10.5 for bug fixes, others 11.x)
  • Tests (if it's testable code, there should be a test for it - get help)
  • Docs (every functionality needs to be documented, see here)
  • Migration incl. install.sql (e.g. if the database schema changes, ...)
  • Upgrade notes (deprecations, important information, migration hints, ...)
  • Label
  • Milestone

@aryaantony92 aryaantony92 merged commit 0508c49 into 10.5 Sep 13, 2022
@aryaantony92 aryaantony92 deleted the admin_escape_values branch September 13, 2022 12:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@aryaantony92 aryaantony92

Labels
Bug
Projects
None yet
Milestone
10.5.6
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@dvesh3 @aryaantony92