Skip to content

docs(api): clarify per-stack API portals and warn about token scope#377

Merged
Matovidlo merged 2 commits into
mainfrom
mvasko/stack-aware-api-portal-links
May 12, 2026
Merged

docs(api): clarify per-stack API portals and warn about token scope#377
Matovidlo merged 2 commits into
mainfrom
mvasko/stack-aware-api-portal-links

Conversation

@Matovidlo
Copy link
Copy Markdown
Contributor

Summary

  • Adds a blue info callout above the "List of Keboola APIs" table making it explicit that all links go to the AWS US portal (api.keboola.com) and that using the wrong stack's portal causes Invalid Token errors
  • Adds a new "API Documentation Portals" subsection under "Stacks and Endpoints" with a table of direct links to each stack's portal (api.*), so users can easily navigate to the right one before testing API calls
  • Renames the scattered "modify the endpoints accordingly" paragraph into a dedicated "Service Endpoints" subsection for better visual separation

Context

Addresses the root cause behind api-service#84: users arriving at the docs from developers.keboola.com had no clear way to find the API portal for their stack, and the existing table links all pointed to AWS US (api.keboola.com) without explanation. Rather than adding a cross-stack selector inside the portal app (which breaks auth), this makes the static docs the authoritative entry point — each stack gets a direct link.

Test plan

  • Preview renders correctly (info callout visible above table, portal table renders under Stacks section)
  • All 5 api.* portal links resolve correctly
  • Anchor #api-documentation-portals works from the info callout link

Add an info callout above the API table explaining the links go to the
AWS US portal and that using the wrong portal causes Invalid Token errors.

Add an "API Documentation Portals" subsection under Stacks and Endpoints
with direct links to each stack's portal (api.*), making it easy to land
on the correct portal before trying out API calls with your token.

Rename "If you are using another stack..." paragraph heading to
"Service Endpoints" to separate it visually from the portal table.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the API overview documentation to make the per-stack nature of the API portal explicit, helping users avoid “Invalid Token” errors caused by using the wrong stack’s portal.

Changes:

  • Adds an info note above the “List of Keboola APIs” table warning that the service links target the US Virginia AWS portal and linking to the per-stack portal list.
  • Introduces an “API Documentation Portals” subsection with direct api.* links per stack.
  • Refactors the stack/endpoints narrative into clearer subsections (“API Documentation Portals” + “Service Endpoints”).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread overview/api/index.md Outdated
Comment thread overview/api/index.md Outdated
Comment thread overview/api/index.md Outdated
… stack name

- Replace kramdown {: .alert} with explicit HTML <div class="alert">
  to match the pattern used by the existing warning alert on the same page
- Scope the note to "api.keboola.com links" — Storage API and Developer
  Portal API link to Apiary, not the per-stack portal
- Rename "AWS US" → "US Virginia AWS" to match the terminology used in
  the Stacks and Endpoints section

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@Matovidlo Matovidlo requested a review from odinuv May 12, 2026 06:54
Copy link
Copy Markdown
Member

@odinuv odinuv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

může bejt

@Matovidlo Matovidlo merged commit 1abd282 into main May 12, 2026
5 checks passed
@Matovidlo Matovidlo deleted the mvasko/stack-aware-api-portal-links branch May 12, 2026 10:33
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.

3 participants