-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Update mintlify setup from mint.json to docs.json #16962
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't go line-by-line through docs.json, but I spot checked the deployment and links seem to work. Is it generated from a script?
The deployment seems fine. Mintlify had a conversion command. Mintlify-scrape appears to have a path issue bug though, which is causing our server docs to not get built and pre-commit to fail. I'm seeing rapid iterations on mintlify-scrape versions. I'll raise with the team. |
cc: @znicholasbrown - saw your PR that updates mint.json to docs.json - if building API server docs will probably need some of what's here - although it's failing ATM, I believe due to Path issues setting working directory with mintlify-scrape or npm. |
Mintlify has been alerted to the issue. After a fix is made, will need to re-run the migration utility to create docs.json and pick up recent changes to mint.json. |
This pull request is stale because it has been open 14 days with no activity. To keep this pull request open remove stale label or comment. |
Allows for more fine-grained control of navigation and is the new way to do things.
Docs build locally in tests.
Generating server API docs has changed a bit. So testing before merge.
Checklist
<link to issue>
"mint.json
.