Skip to content

[UI] Unify Connect UX across run configuration types (plus fix)#3622

Merged
peterschmidt85 merged 2 commits intomasterfrom
feat/ui-task-connect-ports
Mar 4, 2026
Merged

[UI] Unify Connect UX across run configuration types (plus fix)#3622
peterschmidt85 merged 2 commits intomasterfrom
feat/ui-task-connect-ports

Conversation

@peterschmidt85
Copy link
Contributor

@peterschmidt85 peterschmidt85 commented Feb 28, 2026

This PR delivers two major updates and minor UI polish.

  • Major: all run configuration types (dev environments, tasks, services) now have a consistent Connect UI to access the run.
  • Major: fixed incorrect handling of the docker property in launch-form wiring.
  • Minor: styling and interaction polish in Connect/Open flows (button labels, collapsible behavior, link/copy presentation).

…ssues

Add a Connect component for task runs that expose ports, with a two-step
wizard (Attach + Open). Single port shows a simple button; multiple ports
use a ButtonDropdown to select and open any forwarded port.

Also fix two launch wizard issues:
- Add `ports` to the supported YAML fields whitelist
- Stop setting `docker: true` when selecting a Docker image (unrelated field)
- Fix `IJobSpec.app_specs` type from singular to array
- Show `-` for empty configuration path on run details

Made-with: Cursor
@peterschmidt85 peterschmidt85 requested a review from jvstme March 3, 2026 10:40
Address PR feedback by making Connect panels collapsible across run types, using Done to collapse wizard flows, and fixing task Open-step behavior when map_to_port is unset.

Made-with: Cursor
@peterschmidt85 peterschmidt85 changed the title [UI] Add Connect section for tasks with ports [UI] Unify Connect UX across run configuration types (plus fix) Mar 4, 2026
@peterschmidt85 peterschmidt85 merged commit 4b4d1f6 into master Mar 4, 2026
28 checks passed
@peterschmidt85 peterschmidt85 deleted the feat/ui-task-connect-ports branch March 4, 2026 14:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants