Skip to content

Hide platform/contributors from navigation until launch - #57

Closed
SyedFahad7 wants to merge 1 commit into
mainfrom
docs/hide-contributors-nav
Closed

SyedFahad7 wants to merge 1 commit into
mainfrom
docs/hide-contributors-nav

Conversation

@SyedFahad7

Copy link
Copy Markdown
Collaborator

PR #52 noted Contributors is hidden until launch, but platform/contributors was still in the Cloud tab's People group of docs.json, so it shows in the live sidebar.

  • Removes it from navigation only. platform/contributors.mdx stays in the repo (the URL still resolves directly) so relaunching is a one-line revert.
  • No pages link to it, so mint broken-links stays clean.

Copilot AI balanced review requested due to automatic review settings October 26, 2026 20:49

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.

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing.

@mintlify

mintlify Bot commented Oct 26, 2026 •

Copy link
Copy Markdown
Contributor

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

Project Status Preview Updated
graphify-labs-inc 🟢 Ready View Preview Oct 26, 2026, 8:50 PM

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

@graphify-labs-staging graphify-labs-staging Bot 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.

Graphify reviewed this change.

No issues found by static checks — no coupling regressions or blocking issues in the code graph. That is not the same as safe to merge: see what was not checked.

Not checked: tests were not run; no formal proof of the changed code.


Graphify review — findings

Removes the contributors page from the People navigation group in docs.json. The sidebar now shows only the profile entry under People, with no fallback or separate route for contributor listings.

No blocking issues surfaced.

Analysis details — impact, health, verification

Impact & health

Graphify review

Impact — 67 functions depend on the 67 functions this change touches.

Health — grade A; 2 existing hotspot(s) in the area this change touches (pre-existing, not introduced here):

  • instructions — 0 callers, 9 callees (medium)
  • options — 0 callers, 8 callees (medium)

Verification — 67 functions in the blast radius were not formally verified this run (proofs are advisory here).

Gate & verification

graphify gate

PASS — objectively clean (no health regressions, tests not run — proofs not run this pass (advisory)). Grounded, not self-assessed.

Advisory (not blocking):

  • verification_scope: 67 function(s) in the blast radius were not formally verified this run

@SyedFahad7

Copy link
Copy Markdown
Collaborator Author

Closing: not needed. Contributors is intentionally live. #54 reverted #52 and put platform/contributors back in the nav because the production switch is on for every plan.

@SyedFahad7 SyedFahad7 closed this Oct 26, 2026

This branch was successfully deployed

1 active deployment
staging — 7fbf7ce4 Deployed Oct 26, 2026 by mintlify[bot]
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