-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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 API documentation links #18510
Conversation
Updates all previous documentation links with the new structure
Deploying cloudflare-docs with Cloudflare Pages
|
We should probably also update:
cloudflare-docs/public/_redirects Line 647 in 28308f0
cc: @crwaters16 |
Also noticed that API URLs with parentheses aren't included in this PR yet. Example: Probably related to the regex you used to capture existing links? Btw, some of these URLs have escaped parentheses characters in the MDX source. For example: |
Some (apparently normal?) API links still need to be updated — found 174 search results for cloudflare-docs/src/content/docs/cloudflare-one/identity/idp-integration/entra-id.mdx Line 209 in 761df1a
Maybe rerunning the script will fix these occurrences? |
there wasn't a regex used here; it was taking the existing URLs and mapping them to the current documentation URLs. if it isn't picked up, one of those are missing.
this one is due to workaround being used to not escape the parentheses (it's called an automatic link). in the source it is laid out as
while valid, this doesn't get picked up as the URL is wrapped with
these won't automatically get picked up as that is not in the source of the openapi schema which was used for mapping. we can manually update these ones. |
9dc6169
to
e9cb6bb
Compare
e9cb6bb
to
b296093
Compare
b296093
to
928d22a
Compare
🚨 Don't merge 🚨
Updates all previous documentation links with the new API schema site structure.