-
Notifications
You must be signed in to change notification settings - Fork 1.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
Feat/activities custom objects #3213
Conversation
lucasbordeau
commented
Jan 3, 2024
•
edited
edited
- Created a useMultiObjectSearchSelectedItemsQuery hook to replace existing one with multi object querying
- Changed ActivityTargetInlineCellEditMode to accept any custom object
- Created a MultipleObjectRecordSelect component to handle multi object picking
packages/twenty-front/src/modules/ui/layout/selectable-list/components/SelectableItem.tsx
Outdated
Show resolved
Hide resolved
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.
We still need to clean up a bit and I've found a few issues with optimistic rendering but let merge it to move forward
@@ -160,6 +163,7 @@ export const mockedActivities: Array<MockedActivity> = [ | |||
id: '89bb825c-171e-4bcc-9cf7-43448d6fb278t', | |||
createdAt: '2023-04-26T10:12:42.33625+00:00', | |||
updatedAt: '2023-04-26T10:23:42.33625+00:00', | |||
targetObjectNameSingular: 'person', |
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.
Does this actually come from the server?
let relationAlias = fieldMetadata.isCustom | ||
? `${fieldKey}: ${fieldMetadata.targetColumnMap.value}` | ||
? `${fieldKey}: ${referencedObjectMetadata.targetTableName}` |
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.
not sure about this change, I'm facing bugs
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.
I'm facing bug even while putting it back actually.
What I'm doing:
- create a test object
- add a record
- go to that record and try to add an activity
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.
But I think this change should still be reverted and we should check the targetColumnMap value