fix: correct broken anchor, wrong OAuth URL, typos in /overview/ pages - #391
fix: correct broken anchor, wrong OAuth URL, typos in /overview/ pages#391Iamfle4ka wants to merge 1 commit into
Conversation
- Fix OAuth US Virginia GCP URL: europe-west3 → us-east4 (api/index.md) - Fix broken anchor: #regions-and-endpoints → #stacks-and-endpoints (encryption.md) - Fix typo: programatically → programmatically (index.md) - Fix grammar: 'once worked' → 'once you'\''ve worked' (index.md) - Replace absolute URLs with relative paths (api/index.md) Refs: PRDCT-363 Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
keboola-pr-reviewer-bot
left a comment
keboola-pr-reviewer-bot
left a comment
There was a problem hiding this comment.
Verdict: needs_human (risk 2/5) · profile docs
Solid, well-grounded fixes but the OAuth endpoint URL change is a product-behaviour claim I can't confirm, so escalate.
Concerns:
overview/api/index.md: OAuth US Virginia GCP URL change is unverifiable product-behaviour claim
Suggested reviewers: @keboola/docs
|
Superseded by #410, which carries all three files unchanged. Open 43 days with no human review — only Devin, the Linear linkback and `keboola-pr-reviewer-bot`. The bot escalated the one claim here it could not check, and it is now confirmed: the OAuth row for US Virginia GCP pointed at `oauth.europe-west3.gcp.keboola.com`, copy-pasted from the EU Frankfurt row. Branch stays cherry-pickable. |
2 participants
Jira issue(s): PRDCT-363
Changes:
overview/api/index.md): Service endpoint table listedoauth.europe-west3.gcp.keboola.comfor US Virginia GCP — should beoauth.us-east4.gcp.keboola.com(all other services useus-east4.gcpfor that stack; both URLs resolve HTTP 200)overview/encryption.md):#regions-and-endpoints→#stacks-and-endpoints— the heading inapi/index.mdis## Stacks and Endpoints, not## Regions and Endpointsoverview/index.md):programatically→programmaticallyoverview/index.md):once worked your way→once you've worked your wayoverview/api/index.md): Twohttps://developers.keboola.com/...links converted to site-relative pathsUnambiguous fixes only from the
/overview/audit (PRDCT-363). No structural changes, noneeds_humanitems. Full audit log posted as a comment on the Linear issue.Release Notes
Justification, description
Accuracy fixes found during the /overview/ documentation audit: wrong service URL, broken internal anchor, typos, and absolute-to-relative link conversion.
Plans for Customer Communication
N/A
Impact Analysis
N/A — docs-only, no code changes.
Deployment Plan
N/A — merges to master, auto-deployed.
Rollback Plan
N/A
Post-Release Support Plan
N/A
Link to Devin session: https://app.devin.ai/sessions/16d74edde8c441ed8dd6c07a4f7b7f0f
Requested by: @Iamfle4ka