Skip to content

Conversation

ijreilly
Copy link
Collaborator

New Note button was missing because of the wrong condition being applied to display it
Capture d’écran 2025-06-10 à 10 26 57

@ijreilly ijreilly changed the title Fix missins New Note / New Task button Fix missing New Note / New Task button Jun 10, 2025
Copy link
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.

PR Summary

Fixed incorrect permission logic that prevented 'New Note' and 'New Task' buttons from displaying in the UI.

  • Changed condition in packages/twenty-front/src/modules/activities/notes/components/Notes.tsx from !hasObjectUpdatePermissions to hasObjectUpdatePermissions to correctly show the 'New Note' button
  • Applied similar fix in packages/twenty-front/src/modules/activities/tasks/components/TaskGroups.tsx for the 'New Task' button visibility
  • Reorganized imports in TaskGroups.tsx for better code organization
  • Note: Visibility logic between empty/non-empty states may need review for consistency

2 files reviewed, no comments
Edit PR Review Bot Settings | Greptile

Copy link
Contributor

github-actions bot commented Jun 10, 2025

🚀 Preview Environment Ready!

Your preview environment is available at: http://bore.pub:26762

This environment will automatically shut down when the PR is closed or after 5 hours.

@FelixMalfait
Copy link
Member

https://github.com/twentyhq/twenty/blob/main/packages/twenty-front/src/modules/activities/files/components/Attachments.tsx#L106 missed this one too I think?

@ijreilly ijreilly enabled auto-merge (squash) June 10, 2025 09:04
@ijreilly ijreilly merged commit 9cfabd8 into main Jun 10, 2025
52 checks passed
@ijreilly ijreilly deleted the fix-missing-new-note branch June 10, 2025 09:12
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