Skip to content

Improve security docs #1103

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
Merged
merged 34 commits into from
Mar 8, 2025
Merged

Improve security docs #1103

merged 34 commits into from
Mar 8, 2025

Conversation

amirhhashemi
Copy link
Collaborator

  • I have read the Contribution guide
  • This PR references an issue (except for typos, broken links, or other minor problems)

Description(required)

This PR improves the security docs by:

  1. Adding examples.
  2. Improving consistency with other how-to guides.

@bolt-new-by-stackblitz Bolt.new (by StackBlitz)
Copy link

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@netlify Netlify
Copy link

netlify bot commented Mar 1, 2025 •
edited
Loading

❌ Deploy Preview for solid-docs failed. Why did it fail? →

Name Link
🔨 Latest commit 1ff3de3
🔍 Latest deploy log https://app.netlify.com/sites/solid-docs/deploys/67ccbcb03888860008f85220

LadyBluenotes
LadyBluenotes requested changes Mar 8, 2025
View reviewed changes
kodiakhq bot and others added 11 commits March 8, 2025 00:33
@kodiakhq
Merge branch 'main' into improve-security-docs
aa2d138
@kodiakhq
Merge branch 'main' into improve-security-docs
7d1bdca
@amirhhashemi @LadyBluenotes
Update src/routes/solid-start/guides/security.mdx
9e9dcbf 
Co-authored-by: Sarah <gerrardsarah@gmail.com>
@amirhhashemi @LadyBluenotes
Update src/routes/solid-start/guides/security.mdx
25b8d5a 
Co-authored-by: Sarah <gerrardsarah@gmail.com>
@amirhhashemi @LadyBluenotes
Update src/routes/solid-start/guides/security.mdx
b55ce18 
Co-authored-by: Sarah <gerrardsarah@gmail.com>
@amirhhashemi @LadyBluenotes
Update src/routes/solid-start/guides/security.mdx
9ff7ede 
Co-authored-by: Sarah <gerrardsarah@gmail.com>
@amirhhashemi @LadyBluenotes
Update src/routes/solid-start/guides/security.mdx
fd97efa 
Co-authored-by: Sarah <gerrardsarah@gmail.com>
@amirhhashemi @LadyBluenotes
Update src/routes/solid-start/guides/security.mdx
fb7263d 
Co-authored-by: Sarah <gerrardsarah@gmail.com>
@amirhhashemi @LadyBluenotes
Update src/routes/solid-start/guides/security.mdx
930df93 
Co-authored-by: Sarah <gerrardsarah@gmail.com>
@amirhhashemi @LadyBluenotes
Update src/routes/solid-start/guides/security.mdx
21bde21 
Co-authored-by: Sarah <gerrardsarah@gmail.com>
@amirhhashemi
fix grammar and add link to csp in mdn
dc164f9
LadyBluenotes
LadyBluenotes reviewed Mar 8, 2025
View reviewed changes
@amirhhashemi @LadyBluenotes
Update src/routes/solid-start/guides/security.mdx
0660e97 
Co-authored-by: Sarah <gerrardsarah@gmail.com>
@LadyBluenotes LadyBluenotes merged commit 8410839 into solidjs:main Mar 8, 2025
2 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@LadyBluenotes LadyBluenotes LadyBluenotes approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@amirhhashemi @LadyBluenotes