Support all Composite Fields Import

[charlesBochet]

Scope & Context

Table views currently support an "Import" feature (that can be found in the "Option" menu)
This feature enable a user to upload a csv and automatically/manually map the csv columns to record fields.

Current behavior

Currently, the feature does not handle all composite fields (having multiple fieldColumns) properly ; selecting an ADDRESS or a CURRENCY does not work.

Expected behavior

Import feature supports ADDRESS and CURRENCY field types like it supports FULL_NAME field type.

1. Check the behavior of Full_name by trying to make an import on People table. You'll see that fields list is exposing Firstname (Name) + Lastname (Name)
2. We want something similar for Currency: Currency Code (Amount) + Amount Micros (Amount) (Amount being the field name)
3. Similar for Address fields

Technical inputs

The work should be done in: packages/twenty-front/src/modules/object-record/spreadsheet-import/useSpreadsheetRecordImport.ts

Make sure to properly test this hook with composite case. It would also be nice to extract an util to ease testing.

[ijayhub]

good evening, is it still, open? if it is can you assign it to me?

[charlesBochet]

Hi @ijayhub, thanks for looking to contribute, it's much appreciated
The existing PR looks good, we still need to review it and propose a few improvements, no need to open a new one!
They are other good-first-issues available :)

[ijayhub]

Okay... 👍

…

On Mon, 27 May 2024, 8:59 pm Charles Bochet, ***@***.***> wrote: Hi @ijayhub <https://github.com/ijayhub>, thanks for looking to contribute, it's much appreciated The existing PR looks good, we still need to review it and propose a few improvements, no need to open a new one! They are other good-first-issues available :) — Reply to this email directly, view it on GitHub <#5425 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/APYOQHOOWPEVGCEX75LYWW3ZEOGBZAVCNFSM6AAAAABHYGBX2GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMZTHE4TIMRQGU> . You are receiving this because you were mentioned.Message ID: ***@***.***>