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

Block search engines from indexing content on Netlify preview builds #5725

Merged
merged 2 commits into from Feb 1, 2024

Conversation

stevepiercy
Copy link
Collaborator

Copy link

netlify bot commented Feb 1, 2024

Deploy Preview for volto ready!

Name Link
🔨 Latest commit 87ab514
🔍 Latest deploy log https://app.netlify.com/sites/volto/deploys/65bb73d16bcb290008206005
😎 Deploy Preview https://deploy-preview-5725--volto.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.

Copy link

netlify bot commented Feb 1, 2024

Deploy Preview for plone-components canceled.

Name Link
🔨 Latest commit 87ab514
🔍 Latest deploy log https://app.netlify.com/sites/plone-components/deploys/65bb73d1939507000830d060

@stevepiercy
Copy link
Collaborator Author

https://deploy-preview-5725--volto.netlify.app/robots.txt looks good.

No headers got set. I'm looking into why.

@stevepiercy
Copy link
Collaborator Author

Actually the header does get set on Netlify only. I looked in the wrong place:

Screenshot 2024-02-01 at 2 47 50 AM

@ksuess
Copy link
Member

ksuess commented Feb 1, 2024

I wonder if it would be enough in every Netlify "site" but plone/documentation to set the option "branch deploy" to none.

grafik

It remains the preview, which is per Netlify default prevented to be indexed by X-Robots-Tag: noindex.

@stevepiercy
Copy link
Collaborator Author

There is always at least one production deploy, which is either 6.0 for docs or main for volto. Setting it to None would be the default branch. I set both docs and volto to do just that. These PRs are still needed.

@stevepiercy
Copy link
Collaborator Author

@ksuess
Copy link
Member

ksuess commented Feb 1, 2024

Yes, one published deploy always seems to be provided by Netlify. I also do not see a way to prevent every deploy but the preview, which is indeed where handy.
Approving.

@stevepiercy stevepiercy merged commit ecc290e into main Feb 1, 2024
64 checks passed
@stevepiercy stevepiercy deleted the block-netlify-indexing branch February 1, 2024 12:38
sneridagh added a commit that referenced this pull request Feb 1, 2024
* main:
  Block search engines from indexing content on Netlify preview builds (#5725)
  fixed a11y Logo Issue and added Translations (#5722)
  Allow editor to edit metadata name during bulk or single upload (#5560)
  5439 clarify defaultAddonName option and mention to the generator readme the creation of the theme add-on (#5709)
  Remove turbo from monorepo commands until it's really necessary (#5715)
  bugfix: wrong conditional proprieties on ObjectBrowser (#4190)
  Added aria-live='polite' (#5639)
  Release 18.0.0-alpha.9
  Release @plone/scripts 3.3.2
  fix: handle addons that have not been migrated to the new structure o… (#5704)
  Update Semantic UI React to version 2.1.5 (#5632)
  Removed unmaintained and unused razzle-plugin-bundle-analyze (#5671)
  3716 - improve validation of id widget (#4686)
  Fixed redirect of https://tanstack.com/query/v4/docs/react/guides/ssr… (#5700)
  Release 18.0.0-alpha.8
  Release @plone/volto 18.0.0-alpha.7 (not pushed to main)
  Release @plone/slate 18.0.0-alpha.6
  Release @plone/registry 1.2.1 (not pushed to repo)
stevepiercy added a commit that referenced this pull request Mar 24, 2024
stevepiercy added a commit that referenced this pull request Mar 24, 2024
stevepiercy added a commit that referenced this pull request Mar 24, 2024
davisagli pushed a commit that referenced this pull request Mar 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants