Skip to content

Conversation

@erikjanwestendorp
Copy link
Contributor

Description

Add CSP health check article

Fixes: #6336

Type of suggestion

  • Typo/grammar fix
  • Updated outdated content
  • New content
  • Updates related to a new version
  • Other

Product & version (if relevant)

Umbraco > 14.3

Deadline (if relevant)

Thursday, October 3 2024 (release Umbraco 14.3)

@sofietoft
Copy link
Contributor

Thanks for the PR @erikjanwestendorp ! 💪

I'll make sure we get the reviewed and ready for the 14.3 release!

Copy link
Contributor

@sofietoft sofietoft left a comment

Choose a reason for hiding this comment

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

This is great @erikjanwestendorp ! 💪

Could you add the article to the SUMMARY.md file for two versions as well? 🤞 Otherwise it will not show in the published docs.
Also, does it make sense to link to this new article from the Cross-site scripting Protection article, which is currently somewhat empty? 🤔

@erikjanwestendorp
Copy link
Contributor Author

@sofietoft Thanks for the review! 😄 Just updated the PR with your suggestions.

Copy link
Contributor

@sofietoft sofietoft left a comment

Choose a reason for hiding this comment

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

Looks great @erikjanwestendorp !
And thanks for adding it to the 15 docs as well - much appreciated!

I'll get it merged.

@sofietoft sofietoft merged commit 586dda4 into umbraco:main Sep 26, 2024
32 checks passed
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.

Add CSP to health checks article

2 participants