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 lnding page content showing escaped html #1241

Merged

Conversation

pattonwebz
Copy link
Member

What type of PR is this? (check all applicable)

  • 🐛 Bug Fix

Description

This PR swaps a location relying on esc_html to use wp_kses_post instead so that escaped html does not appear in the output of parameterized landing pages.

Related Tickets & Documents

  • Related Issue #
  • Closes #
  • Related Basecamp To-do URL:
  • Related Support Ticket URL:

Proof of Functionality

Accessibility checklist

If your PR includes UI changes, please utilize this checklist:

  • Semantic HTML implemented?
  • Keyboard operability supported?
  • Checked with Accessibility Checker or WAVE and addressed Error and Warning issues?

For more info, check out the
Accessibility Checker Documentation.

Quality assurance

  • I have tested this code to the best of my abilities.
  • I have run local linting and tests.
  • I have checked that the base branch is correctly set.
  • I have updated the documentation or README.md accordingly.

[optional] Are there any post deployment tasks we need to perform?

Additional Notes

@pattonwebz pattonwebz changed the base branch from develop to master June 16, 2024 00:57
@pattonwebz pattonwebz merged commit 80285d5 into master Jun 17, 2024
6 checks passed
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