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

Docs: adds robots.txt notice #1208

Merged
merged 8 commits into from
Jan 30, 2024
Merged

Docs: adds robots.txt notice #1208

merged 8 commits into from
Jan 30, 2024

Conversation

ZPain8464
Copy link
Contributor

@ZPain8464 ZPain8464 requested a review from kenjenkins January 30, 2024 15:31
@ZPain8464 ZPain8464 requested a review from a team as a code owner January 30, 2024 15:31
@ZPain8464 ZPain8464 requested review from cmo-pomerium and removed request for a team January 30, 2024 15:31
Copy link

netlify bot commented Jan 30, 2024

Deploy Preview for pomerium-docs ready!

Name Link
🔨 Latest commit 23a6e87
🔍 Latest deploy log https://app.netlify.com/sites/pomerium-docs/deploys/65b969d96ec87300083080ae
😎 Deploy Preview https://deploy-preview-1208--pomerium-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.


:::

**Public Access** allows all requests to a given route and bypasses authentication and authorization. If you enable this setting, no whitelists should be provided for the route (for example, **Allowed Users**).
Copy link
Contributor

Choose a reason for hiding this comment

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

Also, this feels a little repetitive to me, compared with the previous paragraph. Is there a way to make this a little smoother?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@kenjenkins I revised the text in the last commit. Please let me know if that breaks up the repetitiveness enough.

@ZPain8464 ZPain8464 requested a review from kenjenkins January 30, 2024 21:27
Copy link
Contributor

@kenjenkins kenjenkins left a comment

Choose a reason for hiding this comment

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

Sure, looks good to me.

@ZPain8464 ZPain8464 merged commit 92e195b into main Jan 30, 2024
4 checks passed
@ZPain8464 ZPain8464 deleted the zpain/robots-txt branch January 30, 2024 22:10
backport-actions-token bot pushed a commit that referenced this pull request Jan 30, 2024
* adds robots.txt notice

* runs prettier

* Update content/docs/reference/routes/public-access.mdx

Co-authored-by: Kenneth Jenkins <[email protected]>

* Update content/docs/reference/routes/public-access.mdx

Co-authored-by: Kenneth Jenkins <[email protected]>

* Update content/docs/reference/routes/public-access.mdx

Co-authored-by: Kenneth Jenkins <[email protected]>

* Update content/docs/reference/routes/public-access.mdx

Co-authored-by: Kenneth Jenkins <[email protected]>

* rewrites text

* fixes cspell typo

---------

Co-authored-by: Kenneth Jenkins <[email protected]>
ZPain8464 added a commit that referenced this pull request Jan 30, 2024
Docs: adds robots.txt notice (#1208)

* adds robots.txt notice

* runs prettier

* Update content/docs/reference/routes/public-access.mdx



* Update content/docs/reference/routes/public-access.mdx



* Update content/docs/reference/routes/public-access.mdx



* Update content/docs/reference/routes/public-access.mdx



* rewrites text

* fixes cspell typo

---------

Co-authored-by: zachary painter <[email protected]>
Co-authored-by: Kenneth Jenkins <[email protected]>
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.

2 participants