Documentation quality check: fix gaps in recently changed pages - #7118
Merged
Conversation
Contributor
Author
|
Preview deployment for your docs. Learn more about Mintlify Previews.
|
Contributor
Author
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Workflows to automatically generate PRs for you. |
ethanpalm
approved these changes
Aug 26, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Evaluated the 34 English MDX pages changed in the last day (editor overhaul, new CLI MDX extension page, POC onboarding, llms.txt/MCP updates), fixed the five with the highest-cost gaps, and mirrored every fix into the Spanish, French, and Chinese translations. All corrected facts were verified against the
mintlify/serverandmintlify/mintcodebases.Changes
poc-onboarding.mdx(+es/,fr/,zh/)*.mintlify.appsubdomain. Authentication works on.mintlify.sitesubdomains (verified inclientAuth.controller.tsand consistent with/deploy/authentication-setup). This could push evaluators into unnecessary DNS work.</>icons) that contradicts the overhauled editor docs./editor/publishand/editor/review.editor/index.mdx(+es/,fr/,zh/)/editor/publishand the publish action logic in the dashboard code (feature branches offer Save in branch / Create pull request).deploy/deployments.mdx(+es/,fr/,zh/)/deploy/gitlaband/deploy/bitbucket. Also replaced pre-existing HTML entities with real apostrophes in the French file.cli/commands.mdx(+es/,fr/,zh/)mint configkeys table listed onlysubdomain(used bymint automations), while the same page referencesdateFrom/dateTodefaults formint analytics. Self-contradicting reference.dateFromanddateTorows and expandedsubdomainusage tomint dev,mint analytics,mint add-domain, andmint score(verified inpackages/cli/src/config.tsand command sources).editor/pages.mdx(+es/,fr/,zh/)private-pages-section.tsx).Best-practices pass
es/,fr/, andzh/counterparts, preserving markup,<div id>anchor wrappers, language-prefixed internal links, and each file's existing terminology conventions. Nodocs.jsonnavigation or redirect changes were needed.mint broken-linkspasses; no broken links introduced.editor/pages.mdxneeded a fix (short description, updated in all four languages).Also worth reviewing (not touched)
editor/settings.mdx— Appearance settings (Show all files, Open live preview in new tab, draft-PR default) are undocumented despite inbound links fromeditor/pages.mdx,editor/review.mdx, and the changelog.editor/publish.mdx— no role prerequisite for publishing; "Auto publish" vs settings page's "Main branch autocommits" naming drift; table omits Merge and publish for Mintlify-hosted feature branches.editor/agent.mdx— roles and usage limits for the agent unstated (limits unverifiable from repos).deploy/reverse-proxy.mdx— routing table omits/.well-known/mcp/*,/.well-known/agent-card.json, and/.well-known/api-catalogpaths advertised byai/llmstxt.mdx.ai/mintlify-mcp.mdx— dashboard location of the Publishing toggle not given (exact URL unverifiable).changelog.mdx—/editor/draftslink resolves only via redirect to a page that never mentions drafts.Open gaps left unfixed (unverifiable)
llms.txtsuppresses the generated/_llms/split indexes.Note
Low Risk
Documentation-only corrections and translation parity; no runtime, auth, or deployment behavior changes in this repo.
Overview
Documentation accuracy pass across English and mirrored
es/,fr/, andzh/pages—no product code changes.CLI (
mint config): The configuration-keys table now listsdateFromanddateToformint analytics, and documents which commands usesubdomain(not onlymint automations).Deployments: Auto-deploy setup is described as GitHub-specific (with links to GitLab/Bitbucket), and troubleshooting no longer assumes every customer uses the GitHub App.
Editor: Publishing from a feature branch is clarified—live site updates only after merge to the deployment branch. Private pages docs add what they are, how to create them in the Personal section (replacing the old Private label), and expanded page metadata.
POC onboarding: Step 4 is rewritten for the current editor (toolbar branch creation, keyboard shortcut for visual/MDX, Create pull request preview deployments, merge-to-publish). Spanish, French, and Chinese copies also correct authentication guidance (
.mintlify.sitevs custom subpath) and link to authentication setup.French
deploy/deployments.mdxreplaces HTML-escaped apostrophes with normal characters.Reviewed by Cursor Bugbot for commit d8d7deb. Bugbot is set up for automated code reviews on this repo. Configure here.