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

chore: add redirect from stress testing to grafana page #1507

Merged
merged 5 commits into from
Feb 13, 2024

Conversation

heitortsergent
Copy link
Collaborator

What?

Add a redirect from the stress testing page to the new Grafana blog, to test impact on SEO over canonical.

@heitortsergent
Copy link
Collaborator Author

@ppcano I chatted with @AnneB-SEO2 today about the pillar content project, and some of the results we're seeing, and we wanted to test out if adding a redirect instead of the canonical that we currently have would impact it. We thought we'd test out on one page before adding the redirect to the other Load test type pages.

What do you think?

@ppcano
Copy link
Collaborator

ppcano commented Feb 13, 2024

@heitortsergent,

If I recall well, you have to remove the page in order the redirect to take effect. You can test this in staging.

@heitortsergent
Copy link
Collaborator Author

@ppcano you're right, I deleted the page and I think it should work now.

I was going to just do a test deploy first with the Stress testing page first to make sure it's working, and then set up the rest of the pages in that section, do you think that's ok?

The one thing I'm not sure about is we'll lose that section of the left-hand side menu.

@ppcano
Copy link
Collaborator

ppcano commented Feb 13, 2024

The one thing I'm not sure about is we'll lose that section of the left-hand side menu.

@heitortsergent Yes, I think you will lose it. You can see this running the docs locally.

If you want to include it on the sidebar, you can have an empty md page and set the redirect frontmatter property. See this as an example.

@heitortsergent heitortsergent marked this pull request as ready for review February 13, 2024 20:58
@heitortsergent
Copy link
Collaborator Author

Thanks @ppcano! I'll merge this and see if that helps us with the SEO before I do the rest. 🙏

@heitortsergent heitortsergent merged commit 6ac7e7b into main Feb 13, 2024
4 checks passed
@heitortsergent heitortsergent deleted the chore/add-stress-testing-redirect branch February 13, 2024 21:15
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.

2 participants