Skip to content

feat(workflow): improve Add node discovery#16547

Merged
FelixMalfait merged 2 commits intotwentyhq:mainfrom
adityashirsatrao007:fix/improve-add-node-discovery
Dec 15, 2025
Merged

feat(workflow): improve Add node discovery#16547
FelixMalfait merged 2 commits intotwentyhq:mainfrom
adityashirsatrao007:fix/improve-add-node-discovery

Conversation

@adityashirsatrao007
Copy link
Copy Markdown
Contributor

@adityashirsatrao007 adityashirsatrao007 commented Dec 12, 2025

Description

This PR improves the discoverability of the 'Add node' action in the workflow builder by making it always visible in the action menu.

Changes:

  1. Pin the 'Add node' action - Changed \isPinned\ from \ alse\ to \ rue\ for the \ADD_NODE\ action so it's always visible and easily accessible when viewing a workflow
  2. Unpin 'Add to favorites' and 'Remove from favorites' - These actions are less frequently used in the workflow context, so they've been unpinned to make room for the more relevant 'Add node' CTA

Files Changed:

  • \packages/twenty-front/src/modules/action-menu/actions/record-actions/constants/WorkflowActionsConfig.tsx\

Fixes #16538


Note

Pins ADD_NODE in the workflow action menu and unpins ADD_TO_FAVORITES/REMOVE_FROM_FAVORITES to prioritize workflow-related actions.

  • Workflow action menu config (WorkflowActionsConfig.tsx):
    • Pinning: Set isPinned: true for WorkflowSingleRecordActionKeys.ADD_NODE.
    • Unpinning: Set isPinned: false for SingleRecordActionKeys.ADD_TO_FAVORITES and SingleRecordActionKeys.REMOVE_FROM_FAVORITES under propertiesToOverwrite.

Written by Cursor Bugbot for commit 67b65df. This will update automatically on new commits. Configure here.

- Pin the 'Add node' action in the workflow action menu for better discoverability
- Unpin 'Add to favorites' and 'Remove from favorites' actions for workflows

Fixes #16538
Copilot AI review requested due to automatic review settings December 12, 2025 19:45
Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 1 file

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR enhances the discoverability of the "Add node" action in workflow management by making it always visible in the action menu, while deprioritizing less frequently used favorite-related actions.

  • Changed the ADD_NODE action to be pinned by default for better visibility
  • Unpinned ADD_TO_FAVORITES and REMOVE_FROM_FAVORITES actions to make room for more relevant workflow actions

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown
Member

@FelixMalfait FelixMalfait left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@guillim
Copy link
Copy Markdown
Contributor

guillim commented Dec 15, 2025

CleanShot 2025-12-15 at 11 16 03@2x

IMO we should link to the previous node by default. otherwise we have an orphan node creation, and that is not what users expect

@FelixMalfait
Copy link
Copy Markdown
Member

@guillim I think that goes again the goal of the change which was actually to make branching more discoverable
(@Bonapara fyi)

@FelixMalfait
Copy link
Copy Markdown
Member

didn't realize I hadn't merged. I think we can merge like this for now but if you have better ideas to make branching more discoverable @guillim happy to chat on Discord, I'm also not 100% convinced it's very intuitive yet but still feel it's a step in the right direction!

@FelixMalfait FelixMalfait assigned FelixMalfait and unassigned guillim Dec 15, 2025
@FelixMalfait FelixMalfait merged commit d7b34fc into twentyhq:main Dec 15, 2025
67 of 73 checks passed
@twenty-eng-sync
Copy link
Copy Markdown

Hey @FelixMalfait! After you've done the QA of your Pull Request, you can mark it as done here. Thank you!

@guillim
Copy link
Copy Markdown
Contributor

guillim commented Dec 15, 2025

@guillim I think that goes again the goal of the change which was actually to make branching more discoverable (@Bonapara fyi)

ok fine with me. I was not aware of that behaviour

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve Add node discovery

4 participants