PRDCT-693: put Apps prerequisites in the shared Before you start box - #1147
Merged
Merged
Conversation
The Getting Started box (#1144) becomes a shared component, src/components/Prereqs.astro with its wording in prereqs.mjs and its styles as .prereqs in custom.css, and the markdown twin prints it as text. It gains an id prop, a second list (groupLabel plus <li slot="group">) and a shared kai item. Eight prerequisites blocks on five Apps pages move into it: build-locally (two groups, #before-you-start kept), the four authentication tabs, getting-started, streamlit and streamlit/migrate-to-python-js. The last three become .mdx with unchanged slugs. Also puts {URL} in the Auth0 troubleshooting row in a code span. MDX evaluated it and printed the source of Node's URL class. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
…p dialog A shared StageStrip component draws numbered dots on a line, the Getting Started hub look, with a bold title and one line per stage. It is horizontal in a wide column and vertical in a narrow one, and the markdown twin prints it as a numbered list. build-locally gets one strip per path. The password is in the dialog that Open App opens, not next to Open App. That was checked on a running test app in project 264. build-locally, getting-started, authentication, troubleshooting and reference are corrected, and a screenshot of the dialog goes under Check the app. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…ps-prereqs-box The base replaced the Before you start list's data line with "A table in Storage with the data you want the app to show" and moved What you get by default below Check the app. The box keeps its layout and takes the new wording; the moved section merged on its own. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…-693-apps-stage-strip The base moved What you get by default below Check the app and stopped repeating the password location there. The agent strip stays under the section intro, now above Set up your client. The moved section keeps the base's text, and the password evidence note moves to Check the app, where the password step and the dialog screenshot are. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The merge dropped the old defaults paragraph, and with it the sentence behind the agent strip's fourth stage. The password bullet now has it. The evidence comment also gets a VERIFY(Nikita): the dialog was seen on europe-west3 only, and the UI bundles read between 22 and 29 Sep still had an older App Credentials modal, so its labels need a look on us-east4. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
PRDCT-693: show the Build locally stages as a strip, with the Open app dialog
keboola-pr-reviewer-bot
left a comment
There was a problem hiding this comment.
Verdict: needs_human (risk 4/5) · profile connection-docs
Escalating: the PR changes build/site tooling (src/integrations/page-markdown.mjs, src/styles/custom.css) and adds new components, so it is not a content-bucket PR.
Concerns:
src/integrations/page-markdown.mjs: Build tooling changed; new twin slot-parsing regex needs human review against preview.src/styles/custom.css: ~140 lines of new component CSS; visual output not machine-verifiable.src/content/docs/data-apps/build-locally.mdx: Open-App password dialog may not be deployed on us-east4 yet (VERIFY notes).
Suggested reviewers: @keboola/docs
MiroCillik
reviewed
Sep 30, 2026
MiroCillik
left a comment
Member
There was a problem hiding this comment.
One twin bug and two nits, inline.
- the preview looks right: build-locally at 1280, 1000 and 375 px in light and dark, and authentication at 375 px in dark. No box or strip is wider than its column, and the strip turns vertical at 375 px
- the twins of all five pages print the boxes and strips as plain lists, with no
<Prereqs,<liorslot=left - the removed headings' anchors (
#what-youll-need,#prerequisites, migrate's#before-you-start) have no inbound links undersrc/ - I did not check the Open App dialog on us-east4; the
VERIFY(Nikita)is still open
Member
|
BTW, why check Open app dialog on us-east4 specifically? :) |
Miro's review of #1147. In the markdown twin, an MDX comment inside <Prereqs> or <StageStrip> was glued onto the next item: a group item after it landed in "You need", and a comment on the box's last line leaked into the first stage of the next strip. Comments inside those two lists are now dropped, and the buffer is cleared at both closing tags. No page has such a comment today, and the five pages' twins are unchanged. The Kai page's box now lists the project before Kai, from a new shared item, projectWithKai, with the same wording. The comments in the new files now give the reason for the code rather than its history: no dated quote, no Cloudflare numbers or page counts, no pointers at PR #1144. The TODO about /getting-started/project/ stays. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
… note The note said UI bundles "read between 22 and 29 Sep" still had the old modal. Those were older builds (Last-Modified 25 Aug and 14 Sep); only the logs that named them were from that week. Nothing shows a stack serving the old modal now, so the note says what is known: the dialog was seen on europe-west3, and only a logged-in page shows which build a stack loads. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
GitHub reported a conflict after #1151 and #1152 landed. A fresh merge of main resolved on its own: main's only change to page-markdown.mjs is the llms.txt header line ("apps"), and reference.md keeps #1151's Apps rates and sizes links next to this branch's Open App dialog line. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
MiroCillik
approved these changes
Oct 1, 2026
MiroCillik
left a comment
Member
There was a problem hiding this comment.
All findings fixed in 5307615. The us-east4 Open App dialog VERIFY(Nikita) is still open.
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.
Linear: PRDCT-693 · targets main since #1146 landed · also carries #1149, merged into this branch on 30 September
For the reviewer
The review bot escalated this PR because it changes site code, and its policy always sends that to a person. It asks a person to check three things:
inPrereqsandinStripbranches insrc/integrations/page-markdown.mjs. Compare build-locally with its twin, and authentication with its twin. The box and the strips should come out as plain lists..prereqs*and.stage-strip*blocks insrc/styles/custom.css. Look at the same pages in light and dark and on a phone. The strip turns vertical when the content column is narrower than 36rem.VERIFY(Nikita)marks it.Why
Nikita asked for one look for prerequisites across the docs. Getting Started shows them in a box ("BEFORE YOU START / You need"). The Apps pages used five other forms: an H2 "Before you start", an H2 "What you'll need", an H3 "Prerequisites", and an inline "Before you start." or "Before you start —" inside tabs.
What changed
src/components/Prereqs.astrofor the markup,src/components/prereqs.mjsfor the wording,.prereqs*incustom.cssfor the styles.needs,tabs, and theprompt/ui/clislots.id, for a page that links to the box, andgroupLabel, for a second list whose items are<li slot="group">. The label can't be calledgroup: in MDX a named slot arrives as a prop of the same name and would overwrite it.kai, reuses the wording of the Getting Started Prompt-tab line, checked againstkai/getting-started: Kai is the Kai Agent button on supported stacks, and an organization admin switches it on the first time. Both now come from one constant.projectWithKaiis the Kai page's project line ("on a stack where Kai is available", no sign-up link), so that page lists the project before Kai.page-markdown.mjs) prints the box as text, ported from PRDCT-487: Getting Started rerun on a Free Plan project, with a course-style hub and a 10-second clip #1144, including the second list. Like PRDCT-487: Getting Started rerun on a Free Plan project, with a course-style hub and a 10-second clip #1144's version, it reads the component's props from the opening tag's line, so that tag stays on one line; the component says so in its header. MDX comments inside the box or a strip are dropped from the twin, because a markdown list can't hold them.build-locally: "You need", then "Depending on how you build" (with an AI agent, by hand).#before-you-startstill works. The kbagent setup step in the Cursor, VS Code and ChatGPT app tabs and in From a terminal now links "your stack URL" to the box, as the Claude Code tab already did.authentication: one box in each provider tab.getting-started(Kai),streamlit,streamlit/migrate-to-python-js: renamed from.mdto.mdx, because components need MDX. The slugs are unchanged.opportunity.csvfor readers with no data;{URL}in MDX, which evaluated it, so help.keboola.com shows the source of Node'sURLclass in that bullet. It's a code span now, which MDX leaves alone. The same one-line fix goes to main on its own in PRDCT-692: stop MDX printing JavaScript in the Auth0 troubleshooting row #1148, so it reaches the site sooner./getting-started/project/isn't on main yet. Only build-locally uses it, as before. AVERIFY(Jordan)next to the box asks whether the Free Plan includes apps, which DECISIONS.md on the Getting Started branch lists as open.Also in this PR: #1149
#1149 was merged into this branch, so this PR also carries it. Its description has the detail and the checks.
src/components/StageStrip.astro,.stage-strip*incustom.cssand the twin code draw numbered stages on a line. build-locally gets two strips, one under the intro of "With an AI agent" and one under "By hand".public/data-apps/open-app-dialog.png, sits under Check the app.VERIFYnotes, for things nobody has checked yet:Checks
check-cli-referencehas 0 findings,audit-phase20 broken links and 0 missing images, anddocs-link-redirect-checkagainst PRDCT-693: merge Build with an AI agent into Build an app locally #1146 0 blockers and 0 warnings. Every anchor that points at these pages resolves.docs-link-redirect-checkagainst main has 0 blockers and 0 warnings.<Prereqs,<liorslot=left.#before-you-startlands on the box below the header.Merge order and follow-up
{URL}fix, so whichever lands second has nothing left to change.getting-started/Prereqs.astroandprereqs.mjs;src/components/Prereqs.astro;.gs-prereqs*CSS;page-markdown.mjschanges, keeping their comment stripping and PathIntro text;SHARED.projectback at/getting-started/project/("Get a project").🤖 Generated with Claude Code