PRDCT-551: Storage API under Storage (re-cut off main) - #1053
Merged
Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This was referenced Aug 3, 2026
Iamfle4ka
pushed a commit
that referenced
this pull request
Aug 4, 2026
main moved 118 commits since the baseline was captured, so the gate went red on findings that already exist there (mostly /extend/* links waiting on #1053 / #1054). Re-baselined on the merged tree: 1155 → 1353 known findings, gate back to exit 0, so only new regressions fail. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Iamfle4ka
force-pushed
the
PRDCT-551-storage-api-recut
branch
from
August 4, 2026 23:40
b261626 to
149c9ae
Compare
Re-cut of phase-2 batch 1 (was #1029, stacked on #1027) as a standalone per-unit PR off main, per the 2026-07-27 pivot. Moves the 9 Storage API pages to /storage/api/** (client libraries under /storage/api/clients/), next to the rest of Storage. Old /integrate/storage/* URLs keep working via redirect_from. Build clean (265 pp); audit 0 missing images. Broken links are forward-refs to not-yet-migrated units (/extend/**, /overview/api/, /integrate/jobs/) — heal as those land. Supersedes the stacked #1029 (closed with the #1027 stack once the queue drains). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Full fact-check against the Storage API spec + client repos. Merge itself was drift-free; fixes are the dev source vs current reality: - CRITICAL: R client examples omitted the required `url` arg → every SapiClient$new(token=...) failed. Added url to all examples + fixed the "only a token is required" claim. - R client marked limited-maintenance (last release 2023) with a legacy note pointing to the Python/PHP clients + Storage API. - Fixed stale self-links /integrate/storage/ → /storage/api/clients/ (index, importer) and brought the sample CSV to /storage/api/new-table.csv. - PHP: refreshed the stale install sample (v4.11 → v18.10) + http→https composer. - Python: added `pip install kbcstorage` (PyPI) as the primary install. - configurations: fixed invalid JSON (missing comma in the effective-config example). - TDE exporter: added a legacy-format note (TDE superseded by Hyper) linking the maintained Tableau writers. Component still resolves; support is an owner call. Build clean (265 pp); audit 0 missing images. Remaining broken links are forward-refs to not-yet-migrated units (/extend/**, /integrate/jobs/, /overview/api/). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Iamfle4ka
force-pushed
the
PRDCT-551-storage-api-recut
branch
from
August 4, 2026 23:45
149c9ae to
c1ad886
Compare
This was referenced Aug 19, 2026
Iamfle4ka
pushed a commit
that referenced
this pull request
Aug 19, 2026
…t it at /storage/api/ The row said MERGE → /storage/, but #1053 factually landed the dev page's technical layer (API intro + clients) at /storage/api/ with a clients/ tree. A /storage/ 301 would strand readers one level short of the content. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
The dev Storage page's content (API intro + clients) lives on this page now, but only /integrate/storage/api/ was redirected — the bare URL resolved nowhere. Found by the redirect contract's conservation check (#1091). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…'t lost at retire The dev-side audit consolidation (PRDCT-367, developers-docs#410) fixes four things on pages this PR ports; the port was cut before that PR, so the recut still carried the broken text (the r-client fixes were already in). Applying them here keeps #410 free to merge on dev while the retire stubs win later. - import-export: dead Apiary export link -> api.keboola.com export-async ref - importer: 'avaialable' typo - docker-cli: 'other then' -> 'other than the'; #regions-and-endpoints -> #stacks-and-endpoints (the heading is 'Stacks and Endpoints') - php-client: every new Client([...]) example now passes the required 'url' Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What & why
Re-cut of phase-2 batch 1 (was #1029, stacked on the big-bang #1027) as a standalone per-unit PR off
main, per the 2026-07-27 pivot — same content/placement, no #1027 dependency.Moves the 9 Storage API pages to
/storage/api/**(client libraries under/storage/api/clients/), next to the rest of the Storage docs. Old/integrate/storage/*URLs keep working viaredirect_from.Verification
npm run buildclean — 265 pages.audit-phase2.mjs: 0 missing images. Broken links are forward-refs to not-yet-migrated units (/extend/**from PRDCT-543: Extending Keboola as its own top-level section #1046,/overview/api/,/integrate/jobs/) — heal as those land.Supersedes
Replaces the stacked #1029 (closed with the #1027 stack once the per-unit queue is drained).
🤖 Generated with Claude Code
Placement / dedup + regroup audit (2026-08-03)
Ran the placement/dedup pass (migration-placement skill). No duplication — the existing
storage/tree has no API/importer/exporter/client pages (only unrelated Excel-exportscreenshots under
tables/), sostorage/api/**is net-new and correctly homed under Storage(Storage API is Storage). No stub to replace, no single-page fix needed.
Regroup question for Jordan / Matias — the "API / Developer Reference" home (strong):
The Storage API pages link internally to two pages that don't exist in help —
/overview/api/("API Introduction") and
/integrate/jobs/("Jobs API") — the PRDCT-582 orphans. These same twoare also referenced from Flows and Components. So multiple in-repo API surfaces depend on a shared
API-reference home that isn't there yet. This is a grouping decision, not a single-page fix:
Recommendation: at minimum home the 2 orphans (unblocks the broken links here + on Variables
#1051); whether to also pull Storage/Variables API into a shared group is your call. Not touched
in this PR — surfacing per Jordan's "formulate the question, ask Matias."