-
-
Notifications
You must be signed in to change notification settings - Fork 2.5k
Configure the robots meta tag to allow search and link crawlers #2163
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
Conversation
@defsanmith is attempting to deploy a commit to the Typebot Team on Vercel. A member of the Team first needs to authorize it. |
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR allows users to control the robots meta tag behavior by adding an "allow indexing" option, enabling search engines to index Typebot content when desired.
- Updated settings schema to include an optional boolean property for indexing.
- Modified SEO component to conditionally render the noindex meta tag based on the allow indexing option.
- Updated translation files and MetadataForm to support the new indexing option.
Reviewed Changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.
File | Description |
---|---|
packages/settings/src/schemas.ts | Added optional allowIndexing property to metadata schema. |
apps/viewer/src/components/Seo.tsx | Modified logic to conditionally remove the noindex meta tag when indexing is enabled. |
apps/builder/src/i18n/pt.json, pt-BR.json, fr.json, en.json | Added new translation keys for indexing settings. |
apps/builder/src/features/settings/components/MetadataForm.tsx | Introduced a new SwitchWithLabel for toggling indexing and updated metadata change handlers. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thank you. Can you remove en and pt translations? New code should only provide the en.json translations and then the rest will be translated with Tolgee (https://docs.typebot.io/contribute/guides/translation)
…rench and Portuguese translations
The requested changes have been made. Thank you! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This pull request enables controlling the robots meta tag by introducing an "allow indexing" option. Key changes include adding an allowIndexing field to the metadata schema, updating the SEO component to conditionally remove the noindex tag, and integrating a new switch in the metadata form with corresponding translations.
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
File | Description |
---|---|
packages/settings/src/schemas.ts | Updated schema to include the optional allowIndexing boolean |
apps/viewer/src/components/Seo.tsx | Modified the logic to omit the noindex meta tag when allowIndexing is true |
apps/builder/src/i18n/en.json | Added translations for the allowIndexing label and tooltip |
apps/builder/src/features/settings/components/MetadataForm.tsx | Integrated a new switch toggle for the allowIndexing option |
Configure the robots meta tag to allow search and link crawlers
Description
Closes #1123
This PR adds the "Allow indexing" metadata option. This feature allows users to control whether search engines like Google and LinkedIn can index their Typebot content by removing the
noindex
meta tag when enabled.Changes
MetadataForm.tsx
to include a new switch toggle to allow indexingSeo.tsx
to remove thenoindex
tag when indexing is enableden.json
,fr.json
,pt-BR.json
,pt.json
language files:settings.sideMenu.metadata.allowIndexing.label
: "Allow search engines to index"settings.sideMenu.metadata.allowIndexing.tooltip
: "When enabled, the noindex meta tag will be removed, allowing search engines like Google and LinkedIn to index your typebot."Screenshots