-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
"Name" column is visible on table view but not on kanban view #6427
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR Summary
This PR addresses the visibility issue of the "Name" column in the Kanban view by adjusting the filtering logic in relevant components.
-
packages/twenty-front/src/modules/object-record/record-board/record-board-card/components/RecordBoardCard.tsx
:- Added filtering logic to exclude fields marked as
isLabelIdentifier
fromvisibleFieldDefinitions
.
- Added filtering logic to exclude fields marked as
-
packages/twenty-front/src/modules/object-record/record-board/types/RecordBoardFieldDefinition.ts
:- Introduced an optional boolean property
isLabelIdentifier
to theRecordBoardFieldDefinition
type.
- Introduced an optional boolean property
-
packages/twenty-front/src/modules/object-record/record-index/components/RecordIndexContainer.tsx
:- Removed the filter excluding
isLabelIdentifier
fields fromnewFieldDefinitions
, ensuring the name column is visible in both views.
- Removed the filter excluding
3 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR Summary
This PR addresses the visibility issue of the "Name" column in the Kanban view by adjusting the filtering logic in relevant components.
-
packages/twenty-front/src/modules/object-record/record-board/record-board-card/components/RecordBoardCard.tsx
:- Added filtering logic to exclude fields marked as
isLabelIdentifier
fromvisibleFieldDefinitions
.
- Added filtering logic to exclude fields marked as
-
packages/twenty-front/src/modules/object-record/record-board/types/RecordBoardFieldDefinition.ts
:- Introduced an optional boolean property
isLabelIdentifier
to theRecordBoardFieldDefinition
type.
- Introduced an optional boolean property
-
packages/twenty-front/src/modules/object-record/record-index/components/RecordIndexContainer.tsx
:- Removed the filter excluding
isLabelIdentifier
fields fromnewFieldDefinitions
, ensuring the name column is visible in both views.
- Removed the filter excluding
3 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thank you!
Log
|
This PR was created by GitStart to address the requirements from this ticket: TWNTY-4238.
This ticket was imported from: TWNTY-4238
Description
Demo
https://jam.dev/c/7c9db8e1-9b53-49cf-a44c-0b0fec5b1988
Refs
Fixes #4238