Skip to content

Remove broken FinOps Benchmark nav entry - #347

Merged
ravikiranvm merged 1 commit into
mainfrom
ops-4781
Sep 2, 2026
Merged

Remove broken FinOps Benchmark nav entry#347
ravikiranvm merged 1 commit into
mainfrom
ops-4781

Conversation

@ravikiranvm

Copy link
Copy Markdown
Contributor

Removes the "FinOps Benchmark" nav group from docs.json. It pointed to finops-benchmark/overview, a page that was never merged to main, so https://docs.openops.com/finops-benchmark/overview has served a 404 since April 2026.

The entry slipped into main via #341, which was extracted from the stacked ops-3865 branch and carried over that branch's nav additions without the benchmark page itself. Since the benchmark isn't announced to users yet, the nav group is removed entirely; the page content still lives on ops-3865 and can be revived when the benchmark launches.

Testing

Validated docs.json still parses as JSON after the removal; the block was removed exactly once, no other nav entries touched.

Fixes OPS-4781.

The nav pointed to finops-benchmark/overview, a page that was never
merged to main, so the docs served a 404. The entry slipped in via
PR #341, which was extracted from a stacked branch and carried the
nav change without the page. The benchmark is not announced yet, so
the nav group is removed entirely.

Fixes OPS-4781.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@mintlify

mintlify Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
openops 🟢 Ready View Preview Sep 1, 2026, 11:12 AM

💡 Tip: Enable Workflows to automatically generate PRs for you.

@linear

linear Bot commented Sep 1, 2026

Copy link
Copy Markdown

OPS-4781

Copilot AI left a comment

Copy link
Copy Markdown

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 removes the broken “FinOps Benchmark” navigation group from docs.json, eliminating a docs sidebar link that currently points to a non-existent page and returns a 404.

Changes:

  • Removed the “FinOps Benchmark” nav group that linked to finops-benchmark/overview.
  • Verified there are no remaining references to “FinOps Benchmark” or finops-benchmark/overview in the repo.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@ravikiranvm
ravikiranvm merged commit 3866a8f into main Sep 2, 2026
3 of 4 checks passed
@ravikiranvm
ravikiranvm deleted the ops-4781 branch September 2, 2026 05:18
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