Skip to content

App config form improvements#61

Merged
FluxCapacitor2 merged 7 commits into
mainfrom
ux-fixes
May 10, 2026
Merged

App config form improvements#61
FluxCapacitor2 merged 7 commits into
mainfrom
ux-fixes

Conversation

@skkra0

@skkra0 skkra0 commented May 1, 2026

Copy link
Copy Markdown
Collaborator
  • Redirect to the app page when an app was created, even if the first deployment then fails before build
    • Ensure user can always retry if the first deployment fails before build
  • Show an error if app group name exists
  • Check that a selected workflow can actually be triggered on the selected branch

@FluxCapacitor2

FluxCapacitor2 commented May 10, 2026

Copy link
Copy Markdown
Collaborator

I'm going to leave the last item (checking if the workflow can actually be triggered on the selected branch) out of this PR since it involves a lot of moving parts. It would be somewhat difficult to write a perfectly-accurate matcher that looks at branches and branches-ignore on all the workflow trigger types.

https://docs.github.com/en/actions/reference/workflows-and-actions/workflow-syntax#onpull_requestpull_request_targetbranchesbranches-ignore

@FluxCapacitor2
FluxCapacitor2 marked this pull request as ready for review May 10, 2026 20:25
@FluxCapacitor2
FluxCapacitor2 merged commit 52ff419 into main May 10, 2026
3 checks passed
@FluxCapacitor2
FluxCapacitor2 deleted the ux-fixes branch May 10, 2026 20:28
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