Skip to content

fix(savanna): keep Create API key link on endpoint pages - #176

Merged
Tushar-TG-14 merged 1 commit into
tigergraph:mainfrom
ngarakapati:fix/create-api-key-link
Sep 26, 2026
Merged

Tushar-TG-14 merged 1 commit into
tigergraph:mainfrom
ngarakapati:fix/create-api-key-link

Conversation

@ngarakapati

@ngarakapati ngarakapati commented Sep 25, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • The Authorizations "Create an API key" link on Savanna REST API endpoint pages was resolving to a 404 on the published docs site.
  • The link is hand-written HTML (not an Antora xref), so the publisher rewrote it to a site-root absolute path and dropped the /docs base path.
  • Add Antora's xref page classes so the publisher rewrites it like every other in-content link, and regenerate the endpoint pages.

Test plan

  • Open any endpoint page (e.g. List backups) and click Create an API key in Authorizations
  • Confirm it lands on /docs/savanna/main/rest-api/create-api-key (200), not a 404
  • Confirm nav and overview "create an API key" links still work

Hand-written Authorizations links lacked Antora xref classes, so the
docs publisher rewrote them against the site root and dropped /docs.
@netlify

netlify Bot commented Sep 25, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for thriving-strudel-91d4a9 ready!

Name Link
🔨 Latest commit 3f296c4
🔍 Latest deploy log https://app.netlify.com/projects/thriving-strudel-91d4a9/deploys/6ab669d10667d6000824f386
😎 Deploy Preview https://deploy-preview-176--thriving-strudel-91d4a9.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@Tushar-TG-14
Tushar-TG-14 merged commit aa94bf5 into tigergraph:main Sep 26, 2026
4 of 5 checks passed
@ngarakapati
ngarakapati deleted the fix/create-api-key-link branch September 29, 2026 13:04
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