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

feat: Added nonce docs #10219

Merged
merged 2 commits into from
Dec 27, 2024
Merged

feat: Added nonce docs #10219

merged 2 commits into from
Dec 27, 2024

Conversation

benjackwhite
Copy link
Contributor

Changes

Docs for PostHog/posthog-js#1630

Checklist

  • Words are spelled using American English
  • Titles are in sentence case
  • Feature names are in sentence case too
  • Use relative URLs for internal links
  • If I moved a page, I added a redirect in vercel.json
  • Remove this template if you're not going to fill it out!

Article checklist

  • I've added (at least) 3-5 internal links to this new article
  • I've added keywords for this page to the rank tracker in Ahrefs
  • I've checked the preview build of the article
  • The date on the article is today's date
  • I've added this to the relevant "Tutorials and guides" docs page (if applicable)

Useful resources

Copy link

vercel bot commented Dec 25, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
posthog ✅ Ready (Inspect) Visit Preview Dec 25, 2024 7:12pm

@benjackwhite benjackwhite requested a review from a team December 25, 2024 18:54
@@ -51,6 +51,22 @@ Depending on your compliance needs you can either:

**NOTE**: This list should be enough at the time of writing. As the PostHog application changes rapidly, it is possible that other directives may be needed over time for loading the Toolbar. If you experience issues after implementing one of the above solutions, you can typically debug in the browser tools which part of the CSP is blocking requests.

## Supporting none directives
Copy link
Member

Choose a reason for hiding this comment

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

I'd love for us to have a way to clarify from which posthog-js version onwards this setting exists - same applies to other documentation. I will come up with something tomorrow when Im back in town :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah - this might just get out of hand really quick though

@benjackwhite benjackwhite merged commit c316696 into master Dec 27, 2024
3 checks passed
@benjackwhite benjackwhite deleted the nonce branch December 27, 2024 08:51
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