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.

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

Allow production sites to set the no index meta tag #5635

Closed
dkatzz opened this issue Sep 18, 2023 · 0 comments · Fixed by #5636
Closed

Allow production sites to set the no index meta tag #5635

dkatzz opened this issue Sep 18, 2023 · 0 comments · Fixed by #5636
Assignees
@dkatzz
Labels
bug Something isn't working ce-arkansas deploy Work for the deploy directory or to enable new deployments infrastructure Setting up tooling, deployment, cloud, etc

Comments

@dkatzz
Copy link
Contributor

dkatzz commented Sep 18, 2023

We have a setting STAGING_ADD_NOINDEX_META_TAG that allows sites to be hidden from search results by setting a no index meta tag. However, we currently only allow this setting to work for dev and staging sites, which shouldn't be a restriction.

Additionally, we should make it clear what the default is in our docs and add this flag with the true setting in the example config file, so dev sites that are created are hidden from search by default
@dkatzz dkatzz added bug Something isn't working infrastructure Setting up tooling, deployment, cloud, etc deploy Work for the deploy directory or to enable new deployments ce-arkansas labels Sep 18, 2023
@dkatzz dkatzz self-assigned this Sep 18, 2023
This was referenced Sep 18, 2023
Merged
Merged
Merged
dkatzz added a commit to civiform/civiform-deploy that referenced this issue Sep 18, 2023
@dkatzz
CiviForm deployments created from the example configs should be hidden (
0bc6aa5 
#52)

This change sets the staging_add_noindex_meta_tag flag to true, meaning
a deployment created from this example will be hidden by default to
prevent dev sites from being searchable.

Related to civiform/civiform#5635
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dkatzz dkatzz

Labels
bug Something isn't working ce-arkansas deploy Work for the deploy directory or to enable new deployments infrastructure Setting up tooling, deployment, cloud, etc
Projects
CiviForm Project Roadmap & Tracker
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Allow production sites to set the no index meta tag civiform/civiform
1 participant
@dkatzz