twenty
twenty copied to clipboard
Field on label should change singular to plural based on relation type
Current behavior
- The field on label is populated with active object, and it doesn't change singular to plural or vice versa based on relation type.
Expected behavior
- The label field on should automatically change from singular to plural (or vice versa) based on the relation type, unless the user manually edits or changes the field.
This issue is not part of oss.gg hackathon. Please pick a different one or start with a side quest
/oss.gg 200
Thanks for opening an issue! It's live on oss.gg!
/assign
Assigned to @Cyanhead! Please open a draft PR linking this issue within 48h ⚠️ If we can't detect a PR from you linking this issue in 48h, you'll be unassigned automatically 🕹️ Excited to have you ship this 🚀
/assign
This issue is already assigned to another person. Please find more issues here.
/assign
This issue is already assigned to another person. Please find more issues here.
@Cyanhead, Just a little reminder: Please open a draft PR linking this issue within 12 hours. If we can't detect a PR in 12h, you will be unassigned automatically.
/assign
This issue is already assigned to another person. Please find more issues here.
Hi @ehconitin, I'm having some difficulty with where exactly to implement the fix for this issue. I have an idea of the cause, but i'm not sure where/how to pass the data.
PHOTO A: It looks like the reason the plural is not being applied is because the initialRelationFieldMetadataItem.label gotten from the useRelationSettingsFormInitialValues hook doesn't get assigned the correct value.
PHOTO B: initialRelationType receives relationTypeFromFieldMetadata as undefined and defaults to RelationDefinitionType.OneToMany.
PHOTO C: The SettingsDataModelFieldSettingsFormCard component passes three properties, of which relationDefinition needed by the useRelationSettingsFormInitialValues isn't included.
/assign
This issue is already assigned to another person. Please find more issues here.
/assign
This issue is already assigned to another person. Please find more issues here.
@ehconitin Hello, is this issue still available? Can I work on it?
/assign
This issue is not part of oss.gg hackathon. Please pick a different one or start with a side quest
@FelixMalfait @ehconitin Hello, Sorry for working on this before being assigned. I've completed the task and made a pull request. Please review and let me know if any changes are needed.
Thank you!
@omarNaifer12 no problem! Thanks 😊