PRDCT-692: correct the Apps auto-sleep default and range - #1153
Merged
Merged
Conversation
The reference said the deploy wizard's auto-sleep timeout runs from five minutes to 24 hours with a five-minute default, and operate.md repeated the 24-hour cap. The UI served on europe-west3 on 2026-09-30 (kbc-ui app-D6Vj2Xc8.js) starts the timeout at 900 s, falls back to 900 s when a config has none, rejects values under 300 s, and offers presets up to 14 days plus a Custom field with 0-30 days. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
keboola-pr-reviewer-bot
approved these changes
Oct 1, 2026
keboola-pr-reviewer-bot
left a comment
There was a problem hiding this comment.
Verdict: auto_approve (risk 1/5) · profile connection-docs
Auto-approve: two one-line content edits correcting the Apps auto-sleep timeout range/default to match shipped UI.
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-692
Why
On help.keboola.com/data-apps/reference/, the Sleep and resume section says the deploy wizard's auto-sleep timeout goes from five minutes to 24 hours, with five minutes as the default. Operate and update an app repeats the 24-hour cap. The shipped UI disagrees: five minutes is the minimum, the default is 15 minutes, and the timeout can be set in days. The reference sentence came over from the 2024 Jekyll page ("The default is set to five minutes."), which may have been true at the time.
What changed
One line on each page:
data-apps/reference.md:211: "(five minutes to 24 hours; default five minutes)" now reads "(five minutes to 30 days; default 15 minutes)".data-apps/operate.md:36: "(five minutes to 24 hours, set in the deploy wizard)" now reads "(five minutes to 30 days, set in the deploy wizard)", so the two pages agree.Evidence
The values come from the UI build that europe-west3 served on 30 September. A logged-in project page loaded
kbc-ui/bundle.D8PUgM9Y.js, which importsapp-D6Vj2Xc8.js(Last-Modified 30 Sep, 14:09 GMT), and its chunks are public on ui.keboola-assets.com:CredentialsModal-CaesU65N.js, the deploy modal: the timeout starts atuseState(900), the Deploy buttons are disabled below 300 s, and the hint reads "The minimum timeout is 5 minutes." The timeout control getsallowDays: true.promise-DJWM6Zry.js: when the modal opens, it readsparameters.autoSuspendAfterSeconds, then the olderexpirationAfterHourstimes 3600, and otherwise uses 900 s.TimeoutControl-D0CsWvYQ.js: presets from 15 minutes to 24 hours, then 2, 3, 7 and 14 days, then Custom with Days 0 to 30, Hours 0 to 23 and Minutes 0 to 59.The 14 September build had the same logic. Other sources agree: the screenshot under the reference sentence (July, project 264) shows 15 minutes, kbagent 0.95.0 defaults
--auto-suspendto 900, andbuild-locally.mdxalready says apps sleep after 15 minutes without visitors.Checks
npm run buildis clean (373 pages), and both pages and their markdown twins show the new text.check-cli-referencehas 0 findings.audit-phase2has 0 broken links and 0 missing images; its one item on these pages is the old developers.keboola.com link on the reference page, which PRDCT-658: flip dev-domain links whose targets already live on help #1094 fixes.git merge-tree: PRDCT-693: put Apps prerequisites in the shared Before you start box #1147 (which now carries PRDCT-693: show the Build locally stages as a strip, with the Open app dialog #1149) and PRDCT-694: give Apps their own backend sizes table #1151 merge cleanly with this commit, although PRDCT-693: put Apps prerequisites in the shared Before you start box #1147 also editsoperate.mdnear its top. PRDCT-684: document the keboola-config contract and Python/JS troubleshooting in Apps #1128 and PRDCT-658: flip dev-domain links whose targets already live on help #1094 already conflict with main in other files, andreference.mdauto-merges in both.Heads-up
operate.mdalready says ("keep the current values").🤖 Generated with Claude Code