feat(web): redesign no-connection screen for first-time activation - #321
Conversation
Rebuild the empty state around getting a database connected fast: - Quick connect: paste a redis://, rediss://, or valkey:// URL (or bare host:port) to create a connection in one step; parse failures get actionable errors and connect failures hand off to the prefilled form - Contextual per-route headline and description instead of one generic screen; centered card layout with background accent - Self-hosted: prominent one-click Connect localhost:6379 plus a Docker one-liner; cloud: Create a 1 GB Valkey instance opens the existing provisioning tab with 1gb preselected - Add Connection dialog: pasting a full URL into Host fills every field - Provider guide chips (Upstash, Redis Cloud, ElastiCache, MemoryDB), read-only trust line, and connection troubleshooting link - Entitlement: scale pod memory limit (2x) and PVC size (4x) with the requested maxmemory instead of the fixed chart defaults, and cap maxmemory at 2gb (DTO validator + service guard + sizing clamp) - Docs: quick-connect URL sections in the provider guides
f4a9fba to
7b601e9
Compare
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 7b601e9. Configure here.
| if (!open) { | ||
| setFormData(emptyFormData); | ||
| setTestResult(null); | ||
| setValkeyMaxmemory(null); |
There was a problem hiding this comment.
Dialog close leaves Valkey tab
Medium Severity
Closing the Add Connection dialog only clears form data and valkeyMaxmemory, not addTab. After visiting the Valkey instances tab and dismissing the dialog (X or overlay), Add connection manually reopens on Valkey (default 768mb) instead of the direct connection form.
Reviewed by Cursor Bugbot for commit 7b601e9. Configure here.
|
|
jamby77
left a comment
jamby77
left a comment
There was a problem hiding this comment.
LGTM. Clean URL parser, 2gb cap enforced in depth (DTO + service + sizing clamp), well-tested front and back. Nit above is non-blocking.
2 participants
Summary
Redesigns the "no database connected" screen to convert first-time users: quick connect by pasting a connection URL, one-click localhost connect (self-hosted) or 1 GB instance provisioning (cloud), provider guides, and contextual per-page copy.
Changes
redis:///rediss:///valkey://URL → connection in one step; failures hand off to the prefilled formBefore
After
Checklist
roborev review --branchor/roborev-review-branchin Claude Code (internal)