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

fix(KUI-1378): add noindex robots header for secondary hosts (app.*) #388

Merged
merged 3 commits into from
Jun 26, 2024

Conversation

axelbjo
Copy link
Contributor

@axelbjo axelbjo commented Jun 20, 2024

No description provided.

@axelbjo axelbjo requested a review from belanglos as a code owner June 20, 2024 12:58
@axelbjo axelbjo requested a review from a user June 20, 2024 12:58
Copy link
Contributor

@amirhossein-haerian amirhossein-haerian left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

Copy link
Contributor

@belanglos belanglos left a comment

Choose a reason for hiding this comment

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

Good job! Please just address the comment regarding jest-extended

server/utils/noIndexMiddleware.test.js Show resolved Hide resolved
@axelbjo axelbjo merged commit ad16b25 into master Jun 26, 2024
3 checks passed
@axelbjo axelbjo deleted the issues/KUI-1370-no-indexing branch June 27, 2024 06:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants