-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Show page - disable click for workspace member relation #3464
Show page - disable click for workspace member relation #3464
Conversation
9aeaa09
to
454ae79
Compare
@@ -4,6 +4,54 @@ import { useRelationPicker } from '@/object-record/relation-picker/hooks/useRela | |||
import { IdentifiersMapper } from '@/object-record/relation-picker/types/IdentifiersMapper'; | |||
import { getLogoUrlFromDomainName } from '~/utils'; | |||
|
|||
export const identifierMapper: IdentifiersMapper = ( |
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 did not modified the function. I just moved it out so I could export it in the test
3430dcc
to
db03b00
Compare
@@ -1,10 +1,17 @@ | |||
import { ObjectMetadataItem } from '@/object-metadata/types/ObjectMetadataItem'; | |||
|
|||
const isWorkspaceMember = (objectMetadataItem: ObjectMetadataItem) => | |||
objectMetadataItem.nameSingular === 'workspaceMember'; |
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.
Should I build a type somewhere @charlesBochet @lucasbordeau?
db03b00
to
e94fb36
Compare
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
Disabling click for workspace member relation + adding a test on the component