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

Disallow search engine indexing of hidden event types #3311

Closed
PeerRich opened this issue Jul 11, 2022 · 6 comments · Fixed by #3478
Closed

Disallow search engine indexing of hidden event types #3311

PeerRich opened this issue Jul 11, 2022 · 6 comments · Fixed by #3478
Labels
✅ good first issue Good for newcomers 🙋🏻‍♂️help wanted Help from the community is appreciated 🧹 Improvements Improvements to existing features. Mostly UX/UI

Comments

@PeerRich
Copy link
Member

Discussed in https://github.com/calcom/cal.com/discussions/3285

Originally posted by GlitchWitch July 5, 2022

Is your proposal related to a problem?

When creating a hidden event type, there is the chance that if the link is shared that it will be crawled and indexed by search engines.

Describe the solution you'd like

When marking an event type as hidden, we should add <meta name="robots" content="noindex"> to the page html header to prevent indexing.

Describe alternatives you've considered

Another possible solution is to add a checkbox to allow the user to decide if the event type can be indexed or not, however I think just linking this to the hidden parameter is good enough.

Additional context

One of my hidden event types currently shows up in search engines as I share it on a public post with a specific CTA and purpose. However I have deliberately chosen to make this event type hidden and do not want it indexed.

@PeerRich PeerRich added ✅ good first issue Good for newcomers 🙋🏻‍♂️help wanted Help from the community is appreciated 🧹 Improvements Improvements to existing features. Mostly UX/UI labels Jul 11, 2022
@gracebir
Copy link
Contributor

I am going to take a look on

@PeerRich
Copy link
Member Author

I am going to take a look on

thank you!

@PeerRich
Copy link
Member Author

cc @GlitchWitch I reopened it here

@MuriloAmarals
Copy link
Contributor

MuriloAmarals commented Jul 11, 2022

Hey @gracebir @PeerRich i've already resolved!
Even today gitstart will send the PR of the resolution.

@gracebir
Copy link
Contributor

Hey @gracebir @PeerRich i've already resolved! Even today gitstart will send the PR of the resolution.

sure

@PeerRich
Copy link
Member Author

PeerRich commented Jul 11, 2022

Hey @gracebir @PeerRich i've already resolved! Even today gitstart will send the PR of the resolution.

awesome, feel free to link the PR here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✅ good first issue Good for newcomers 🙋🏻‍♂️help wanted Help from the community is appreciated 🧹 Improvements Improvements to existing features. Mostly UX/UI
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants