Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Avoid unnecessarily early escape of Site Health check labels#332

Merged
felixarntz merged 3 commits into
WordPress:release/1.1.0WordPress/performance:release/1.1.0from
mukeshpanchal27:fix/correct-escapingCopy head branch name to clipboard
May 10, 2022
Merged

Avoid unnecessarily early escape of Site Health check labels#332
felixarntz merged 3 commits into
WordPress:release/1.1.0WordPress/performance:release/1.1.0from
mukeshpanchal27:fix/correct-escapingCopy head branch name to clipboard

Conversation

@mukeshpanchal27

@mukeshpanchal27 mukeshpanchal27 commented May 10, 2022

Copy link
Copy Markdown
Member

As suggested by @felixarntz in the #329 ticket.

Checklist

  • PR has either [Focus] or Infrastructure label.
  • PR has a [Type] label.
  • PR has a milestone or the no milestone label.

@felixarntz felixarntz changed the title Correct escaping function Avoid unnecessarily early escape of Site Health check labels May 10, 2022

@felixarntz felixarntz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Great, thank you @mukeshpanchal27!

@felixarntz
felixarntz requested a review from tillkruss May 10, 2022 17:52
@felixarntz felixarntz added [Type] Enhancement A suggestion for improvement of an existing feature Infrastructure Issues for the overall performance plugin infrastructure labels May 10, 2022
@felixarntz felixarntz added this to the 1.1.0 milestone May 10, 2022
@felixarntz
felixarntz changed the base branch from trunk to release/1.1.0 May 10, 2022 19:52
@felixarntz

Copy link
Copy Markdown
Member

@mukeshpanchal27 I've updated the base branch to release/1.1.0, with the goal to get this into the upcoming release.

Can someone else please approve this by tomorrow? cc @adamsilverstein @manuelRod @kirtangajjar @mitogh @tillkruss 🙌

@felixarntz
felixarntz merged commit 6e72e47 into WordPress:release/1.1.0 May 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Infrastructure Issues for the overall performance plugin infrastructure [Type] Enhancement A suggestion for improvement of an existing feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.