Skip to content

fix: initialize action argument values from defaults in node modal - #257

Merged
RyanTheRobothead merged 1 commit into
unstablefrom
fix/190-node-modal-checkbox-defaults
Mar 27, 2026
Merged

fix: initialize action argument values from defaults in node modal#257
RyanTheRobothead merged 1 commit into
unstablefrom
fix/190-node-modal-checkbox-defaults

Conversation

@RyanTheRobothead

@RyanTheRobothead RyanTheRobothead commented Mar 27, 2026

Copy link
Copy Markdown
Member

Summary

Details

The ArgumentDefinition objects from the backend include a default field, but the frontend never used it to initialize the editable value field. This caused boolean checkboxes to always appear unchecked regardless of the action's declared default. The fix also benefits text and number fields by pre-filling them with their defaults.

Test plan

  • Open the dashboard and navigate to a node modal with boolean action arguments
  • Verify checkboxes reflect the correct default values (checked if default: true, unchecked if default: false)
  • Verify text and number fields also show their default values pre-filled
  • Verify that manually changing a value still works correctly
  • Verify "Send Action" and "Show Copyable Workflow Step" still use the correct values

Closes #190

🤖 Generated with Claude Code

)

Checkboxes (and other form fields) on the node modal now reflect the
default values defined in the action's ArgumentDefinition. Previously,
item.value was never initialized from item.default, so boolean checkboxes
always rendered unchecked regardless of the action's declared default.

Also removes the redundant :value prop from v-checkbox, which is
unnecessary when v-model with :true-value/:false-value is used.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown

Coverage report

This PR does not seem to contain any modification to coverable code.

@RyanTheRobothead
RyanTheRobothead merged commit 678755f into unstable Mar 27, 2026
4 checks passed
@RyanTheRobothead
RyanTheRobothead deleted the fix/190-node-modal-checkbox-defaults branch April 9, 2026 19:47
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.

1 participant