EPMRPP-114802 || Implement canonical and robots meta tags for duplicated Docs case-study pages#1112
Conversation
* Update pytest.md * Merge pull request #1104 from reportportal/feature/EPMRPP-114901-Add-a-trigger-for-RP-docs-rebuild-on-release-creation-update EPMRPP-114901 || Add a trigger for RP docs rebuild on release creation/update --------- Co-authored-by: Vadzim Hushchanskou <HardNorth@users.noreply.github.com>
…PMRPP-114802-canonical-and-robots-meta-tags
WalkthroughThis PR updates four case study MDX files with consistent metadata and formatting improvements. Each file receives a structured frontmatter title and description, document head sections with SEO metadata (robots tag, canonical URL), and reformatted MediaViewer components to explicit multiline JSX syntax. ChangesCase Study Documentation Enhancement
Estimated Code Review Effort🎯 2 (Simple) | ⏱️ ~8 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
docs/case-studies/ReducingRegressionTimeBy50.mdx (1)
22-22:⚠️ Potential issue | 🟡 Minor | ⚡ Quick winTypo: "Unavailble" → "Unavailable"
✏️ Proposed fix
- - Unavailble environments (15VMs) blocked by aggregation scripts + - Unavailable environments (15VMs) blocked by aggregation scripts🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@docs/case-studies/ReducingRegressionTimeBy50.mdx` at line 22, Fix the typo "Unavailble" to "Unavailable" in the case study text string inside docs/case-studies/ReducingRegressionTimeBy50.mdx (the line that currently reads "Unavailble environments (15VMs) blocked by aggregation scripts"); update that phrase to "Unavailable environments (15 VMs) blocked by aggregation scripts" making sure to correct the misspelling and optionally add a space in "15VMs" to "15 VMs" for readability.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@docs/case-studies/ReducingRegressionTimeBy50.mdx`:
- Around line 8-14: The MDX file inserts a <head> block with a <link
rel="canonical"> which may duplicate Docusaurus's auto-generated canonical; open
the rendered reducing-regression-time-by-50 page, inspect the document <head>,
and confirm there is exactly one <link rel="canonical"> pointing to
reportportal.io; if you find two, remove the manual <head> canonical from
ReducingRegressionTimeBy50.mdx (or the other three MDX files) and instead
override the theme-level SiteMetadata/Head component (swizzle Head or
SiteMetadata) to emit the single canonical you need so Docusaurus does not also
output a self-referencing tag.
---
Outside diff comments:
In `@docs/case-studies/ReducingRegressionTimeBy50.mdx`:
- Line 22: Fix the typo "Unavailble" to "Unavailable" in the case study text
string inside docs/case-studies/ReducingRegressionTimeBy50.mdx (the line that
currently reads "Unavailble environments (15VMs) blocked by aggregation
scripts"); update that phrase to "Unavailable environments (15 VMs) blocked by
aggregation scripts" making sure to correct the misspelling and optionally add a
space in "15VMs" to "15 VMs" for readability.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 428601b3-2cd2-417e-9528-31fa4c1186b6
📒 Files selected for processing (4)
docs/case-studies/ImprovingTestAutomationStability.mdxdocs/case-studies/IncreasingTestAutomationStabilityAndVisibility.mdxdocs/case-studies/ReducingRegressionAnalysisEfforts.mdxdocs/case-studies/ReducingRegressionTimeBy50.mdx
* Update GitHub workflows * EPMRPP-115059 || Remove Analyzer train mention from the docs (#1109) Co-authored-by: Yuliya_Prihodko <Yuliya_Prihodko@epam.com> * EPMRPP-114977 || Exclude /docs/search/ page from indexing and sitemap (#1108) * docs: update case studies with titles, meta tags, and formatting adjustments (#1112) Co-authored-by: maria-hambardzumian <164881199+maria-hambardzumian@users.noreply.github.com> Co-authored-by: Yuliya Prihodko <107035036+pressayuliya@users.noreply.github.com> Co-authored-by: Yuliya_Prihodko <Yuliya_Prihodko@epam.com> Co-authored-by: Vadzim Hushchanskou <HardNorth@users.noreply.github.com> --------- Co-authored-by: maria-hambardzumian <maria_hambardzumyan@gmail.com> Co-authored-by: Yuliya Prihodko <107035036+pressayuliya@users.noreply.github.com> Co-authored-by: Yuliya_Prihodko <Yuliya_Prihodko@epam.com> Co-authored-by: Aliaksei Rudak <143392025+a-rudak-EIS@users.noreply.github.com> Co-authored-by: Vadzim Hushchanskou <HardNorth@users.noreply.github.com>
EPMRPP-114802 || Implement canonical and robots meta tags for duplicated Docs case-study pages
Summary
Add
noindex,followrobots meta tag and custom canonical URLs to all 4 case study pages to point search engines to the canonical versions onreportportal.io(outside the/docs/path).Changes
titlefrontmatter to ensure the page title is correctly set<head>JSX block withrobotsmeta tag and customlink rel="canonical"pointing to the corresponding pages onreportportal.ioPages affected
docs/case-studies/ReducingRegressionTimeBy50.mdxdocs/case-studies/IncreasingTestAutomationStabilityAndVisibility.mdxdocs/case-studies/ReducingRegressionAnalysisEfforts.mdxdocs/case-studies/ImprovingTestAutomationStability.mdxSummary by CodeRabbit
Release Notes