Skip to content

fix(DesignerV2): Added forceSave on run button, fixed connection panel issue (v5.961)#9201

Merged
rllyy97 merged 2 commits into
hotfix/v5.961from
rileyevans/fix/designer-v2-workflow-state-cleanup-hotfix
May 20, 2026
Merged

fix(DesignerV2): Added forceSave on run button, fixed connection panel issue (v5.961)#9201
rllyy97 merged 2 commits into
hotfix/v5.961from
rileyevans/fix/designer-v2-workflow-state-cleanup-hotfix

Conversation

@rllyy97
Copy link
Copy Markdown
Contributor

@rllyy97 rllyy97 commented May 20, 2026

Commit Type

  • fix - Bug fix

Risk Level

  • Low - Minor changes, limited scope

What & Why

Added forceSave option to the FloatingRunButton component.
Fixed some standalone v2 issues.
Fixed some connection panel render issues.

Impact of Change

  • Users: Connections panel action list now renders icons inline and left-aligned instead of overlapping/centered
  • Developers: FloatingRunButton accepts a new forceSave prop; getConnectionConfiguration manifest param is now optional
  • System: Cleaner separation between designer identity (designerID) and workflow content state (WorkflowJson)

Test Plan

  • Unit tests added/updated
  • E2E tests added/updated
  • Manual testing completed
  • Tested in: Standalone designer (Standard + Consumption), connections panel with multiple connectors

Contributors

@rllyy97

Screenshots/Videos

N/A

rllyy97 added 2 commits May 20, 2026 15:06
…oatingRunButton

- Use WorkflowJson instead of Workflow type for standalone designer state
- Remove id spreading from workflow state updates (use separate designerID)
- Add forceSave prop to FloatingRunButton to allow saving even when not dirty
- Fix getConnectionConfiguration manifest parameter to be optional
- Simplify monitoring view workflow restoration
…lignment

- Remove position:absolute from action icon in NodeLinkButton (both v1 and v2)
- Left-align button content with justifyContent: flex-start
@rllyy97 rllyy97 added the risk:low Low risk change with minimal impact label May 20, 2026
@rllyy97 rllyy97 changed the title fix(DesignerV2): Added forceSave on run button, fixed connection panel issue (v5.961) fix(DesignerV2): Added forceSave on run button, fixed connection panel issue (v5.961) May 20, 2026
@github-actions
Copy link
Copy Markdown
Contributor

❌ PR Validation Error

An error occurred while validating your PR. Please try again later or contact the maintainers.

Error: API request failed with status 500

1 similar comment
@github-actions
Copy link
Copy Markdown
Contributor

❌ PR Validation Error

An error occurred while validating your PR. Please try again later or contact the maintainers.

Error: API request failed with status 500

@rllyy97 rllyy97 enabled auto-merge (squash) May 20, 2026 20:18
@github-actions
Copy link
Copy Markdown
Contributor

❌ PR Validation Error

An error occurred while validating your PR. Please try again later or contact the maintainers.

Error: API request failed with status 500

@github-actions
Copy link
Copy Markdown
Contributor

📊 Coverage Check

The following changed files need attention:

libs/designer-v2/src/lib/ui/panel/connectionsPanel/allConnections/nodeLinkButton.tsx - 0% covered
libs/designer/src/lib/ui/panel/connectionsPanel/allConnections/nodeLinkButton.tsx - 0% covered

Please add tests for the uncovered files before merging.

@rllyy97 rllyy97 merged commit dc82dde into hotfix/v5.961 May 20, 2026
11 of 16 checks passed
@rllyy97 rllyy97 deleted the rileyevans/fix/designer-v2-workflow-state-cleanup-hotfix branch May 20, 2026 20:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk:low Low risk change with minimal impact

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants