Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tasks assigned to me view (#9567) #9568

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

eliasylonen
Copy link
Contributor

Issue: twentyhq/core-team-issues#154

View is now added.

TODO: Fix filtering a relation field by 'Me' in Tasks.

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

Added a new 'Tasks assigned to me' table view with filtering for tasks assigned to the current workspace member, and adjusted the position of existing task views.

  • Added /packages/twenty-server/src/engine/workspace-manager/standard-objects-prefill-data/views/tasks-assigned-to-me.ts with table configuration and 'Me' filter
  • Modified position of tasks-by-status view from 0 to 1 in tasks-by-status.view.ts to accommodate new view
  • Added tasksAssignedToMeView to view definitions in seed-view-with-demo-data.ts
  • Known issue: Filtering relation fields by 'Me' in Tasks needs fixing

3 file(s) reviewed, 3 comment(s)
Edit PR Review Bot Settings | Greptile

Copy link
Member

@charlesBochet charlesBochet left a comment

Choose a reason for hiding this comment

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

Thanks @eliasylonen, we will also need an upgrade command for the existing workspaces :)

@eliasylonen
Copy link
Contributor Author

@charlesBochet Upgrade command works now! Ready for review

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.

3 participants