Conversation
...count/authentication-methods/confirm-the-enrollment-of-a-supported-authentication-method.mdx
Outdated
Show resolved
Hide resolved
|
@nlucero I don't know that the console errors are related to this, but I am still trying to identify a root cause and see if I can address. |
This issue is fixed. |
| const mdxContent = dedent` | ||
| --- | ||
| openapi: ${file} ${method} ${path} | ||
| openapi: ${method} ${path} |
There was a problem hiding this comment.
Question: This doesn't seem right. According to their docs:
If you have multiple OpenAPI specifications, include the file path in your reference.
Since we will eventually have multiple OASs, shouldn't we be explicit here and set the file property as well?
There was a problem hiding this comment.
I don't believe this is an issue when you declare the OAS files in docs.json (ref)
Locally I added a second My Account API + a sample Petstore API, generated the files, and things worked as expected.
There was a problem hiding this comment.
Will this also work with different localized versions of the same schema? E.g. when viewing a MyAccount endpoint page in fr-ca, it should read from main/docs/oas/myaccount/myaccount-api-oas.fr-ca.json.
There was a problem hiding this comment.
✋🏽 Change Request: Please revert this change. The openapi property needs three values for the scenario we are creating for ourselves. We need file, method, and path
(see prev comment - ref)
| return docsJson; | ||
| } | ||
|
|
||
| async function getOasFilePath({ locale, oasConfig }) { |
There was a problem hiding this comment.
Question: Where are we using this function? I only see it imported in tests 🤔
There was a problem hiding this comment.
Good catch! This was a miss on my part when I was testing the English fallback 🙈 . This has been reimplemented!
| if (locale === "en") { | ||
| return enPath; | ||
| } | ||
| const localeFilename = chain(oasConfig.outputFile) |
There was a problem hiding this comment.
Nit: Add a code comment above this line with a little example of what this does, it should help when reading it.
| @@ -0,0 +1,9 @@ | |||
| --- | |||
| openapi: post /authentication-methods/{authentication_method_id}/verify | |||
There was a problem hiding this comment.
✋🏽 Change Request: The openapi property should have three values: file, method, path. We'll need to update the generation script to add these.
| "homepage": "https://github.com/auth0/docs-v2#readme", | ||
| "dependencies": { | ||
| "@a0/myaccount-api-oas": "1.404.0", | ||
| "@fern-api/auth0-myaccount-snippets": "^0.0.2", |
There was a problem hiding this comment.
✋🏽 Change Request: can you please pin this dependency
| --- | ||
|
|
||
| import { ReleaseLifecycle } from "/snippets/ApiReleaseLifecycle.jsx"; | ||
| import { ReleaseLifecycle } from "/snippets/ReleaseLifecycle.jsx"; |
There was a problem hiding this comment.
✋🏽 Change Request: The snippet should be called ApiReleaseLifecycle. Can we update the name of the file instead of changing the name here please.
| const mdxContent = dedent` | ||
| --- | ||
| openapi: ${file} ${method} ${path} | ||
| openapi: ${method} ${path} |
There was a problem hiding this comment.
✋🏽 Change Request: Please revert this change. The openapi property needs three values for the scenario we are creating for ourselves. We need file, method, and path
(see prev comment - ref)
| const indexContent = dedent` | ||
| --- | ||
| title: "${apiName}" | ||
| description: "${apiName} endpoints" |
There was a problem hiding this comment.
✋🏽 Change Request: there shouldn't be untranslated text in these templates. This value can just be passed in.
| * Note: This file will be ignored if it already exists to prevent | ||
| * loss of any changes. | ||
| */ | ||
| async function writeApiIndexMdx({ docpath, apiName }) { |
There was a problem hiding this comment.
❓ Question: How do we feel about making this more clear and creating a title, and description configuration. This way we can change the values that get passed in (for translations or otherwise).
| } catch { | ||
| // File doesn't exist, create it | ||
| try { | ||
| await fs.writeFile(indexFilePath, indexContent); | ||
| } catch (err) { | ||
| console.error(`failed to write index: ${indexFilePath}`, err); | ||
| } | ||
| } |
There was a problem hiding this comment.
🤔 This can technically fail for user access reasons. What do you think about using stat instead? The only reason fs.stat throws is if the file doesn't exist.
| docsJson.navigation.languages[langIdx].tabs[refIdx].dropdowns.push({ | ||
| dropdown: oasConfig.docSectionNameMap[locale], | ||
| icon: "list", | ||
| openapi: specPath, |
There was a problem hiding this comment.
✋🏽 Change Request: I think we're going to need to remove this as when this property is included the sidebar structure is not what we're expecting.
Removing this, means that we don't require
specPathat multiple locations, we can probably remove thegetOasFilePathfunction, which removes the need forpatchDocsJsonto be anasyncfunction.
| // and ensure the openapi field is always present | ||
| docsJson.navigation.languages[langIdx].tabs[refIdx].dropdowns[ | ||
| apiIdx | ||
| ].pages = [`${docsPath}/index`, docsByLocale[locale]]; | ||
| ].openapi = specPath; |


Description
This PR begins the process of implementing MyAccount API in Mintlify.
Changes:
docs.jsonReferences
https://auth0team.atlassian.net/browse/DM-289
Testing
Screen.Recording.2026-02-20.at.2.15.27.PM.mov
Screen.Recording.2026-02-19.at.4.34.28.PM.mov
Checklist
CONTRIBUTING.md.