Skip to content

PRDCT-551: Storage API under Storage (re-cut off main) - #1053

Merged
Iamfle4ka merged 7 commits into
mainfrom
PRDCT-551-storage-api-recut
Aug 26, 2026
Merged

Iamfle4ka merged 7 commits into
mainfrom
PRDCT-551-storage-api-recut

Conversation

@Iamfle4ka

@Iamfle4ka Iamfle4ka commented Jul 28, 2026 •

Copy link
Copy Markdown
Collaborator

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 via redirect_from.

Verification

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-export
screenshots under tables/), so storage/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 two
are 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:

BEFORE (today — homeless, broken forward-refs)
  Storage › Storage API            → links to /overview/api/  (404)
  Transformations › … › Variables API → links to /integrate/jobs/ (404)
  Flows, Components                → link to the same two (404 / external dev-docs)

OPTION A — home each orphan under its nearest product surface
  Overview › API Introduction      (/overview/api/)
  Storage  › Jobs API              (/integrate/jobs/ → e.g. /storage/jobs/api or /components/…)
  (Storage API + Variables API stay under their products)

OPTION B — a small shared "API / Developer Reference" group (recommended to consider)
  API Reference
    ├─ API Introduction  (/overview/api/)
    ├─ Jobs API          (jobs run/polling)
    ├─ Storage API       (move or alias)
    └─ Variables API     (move or alias)
  → one home the product pages link into; matches the docs.github "reference" pattern

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."

@vercel

vercel Bot commented Jul 28, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
connection-docs Ready Ready Preview Aug 26, 2026 1:29pm

Request Review

@linear-code

linear-code Bot commented Jul 28, 2026

Copy link
Copy Markdown

PRDCT-551

@Iamfle4ka Iamfle4ka added the dev-docs-migration developers.keboola.com → help.keboola.com migration label Jul 28, 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
Iamfle4ka force-pushed the PRDCT-551-storage-api-recut branch from b261626 to 149c9ae Compare August 4, 2026 23:40
Nikita and others added 2 commits August 5, 2026 01:42
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
Iamfle4ka force-pushed the PRDCT-551-storage-api-recut branch from 149c9ae to c1ad886 Compare August 4, 2026 23:45
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>

@davidesner davidesner left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@Iamfle4ka
Iamfle4ka merged commit a9c07f2 into main Aug 26, 2026
3 checks passed
@Iamfle4ka
Iamfle4ka deleted the PRDCT-551-storage-api-recut branch August 26, 2026 13:31

This branch was successfully deployed

1 active deployment
Preview — afc4f5ba Deployed Aug 26, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dev-docs-migration developers.keboola.com → help.keboola.com migration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants