feat: get extrafields form the API

[Volubyl]

rel #562

Cette PR :

• ajoute les extra_fields à la réponse de la route /datasets/filters si un catalog_id est renseigné comme query param

[Volubyl]

@florimondmanca c'est ici que j'ai le plus de mal.

De ce que je comprends tous les handlers ne retournent pas des entities mais des view. C'est bien ça ?

Donc ma logique était de créer une ExtraFieldView.

Je m'interroge sur le meilleur moyen de passer d'un Extra_Field à un Extra_FieldView .

Est-ce que le parse_extra_fields est le bon moyen ?

Ma première approche était de transformer un ExtraField en ExtraFieldView en faisant un mapping manuel qui aurait ressemblé à :

ExtraFieldView(
title = extra_field.title,
name = extra_field.name,
....
)

Sauf que ExtraFieldView n'est pas instanciable.

Donc j'ai recopié ce qui se faisant dans le fichier d'entitée de l'ExtraField

De ce que je comprends en lisant la doc de parse_obj_as c'est une fonction qui permet de convertir n'importe quelle classe en un type donné. Ici ExraFieldView

Est-ce que c'est la bonne approche ?

[florimondmanca]

Je pense que tu pourrais définir la view explicitement, non ? "Qu'est-ce que je veux renvoyer à mon API"

class ExtraFieldView(BaseModel):
    type: ExtraFieldType
    name: str
    title: str
    # Etc...
    data: dict

Ensuite tu les construis explicitement dans le handler de GetAllExtraFields

async def get_all_extra_fields(query: GetAllExtraFields) -> List[ExtraFieldView]:
    repository = resolve(ExtraFieldRepository)
    extra_fields = await repository.get_all(query.organization_id)

    return [
        ExtraFieldView(
            type=extra_field.type,
            name=extra_field.name,
            title=extra_field.title,
            # Etc...
            data=extra_field.data,
        ) for extra_field in extra_fields
    ]

[florimondmanca]

Dans l'esprit de ce qu'est name, ici tu pourrais utiliser "my_extra_field"

[florimondmanca]

Je pense que tu pourrais définir la view explicitement, non ? "Qu'est-ce que je veux renvoyer à mon API"

class ExtraFieldView(BaseModel):
    type: ExtraFieldType
    name: str
    title: str
    # Etc...
    data: dict

Ensuite tu les construis explicitement dans le handler de GetAllExtraFields

async def get_all_extra_fields(query: GetAllExtraFields) -> List[ExtraFieldView]:
    repository = resolve(ExtraFieldRepository)
    extra_fields = await repository.get_all(query.organization_id)

    return [
        ExtraFieldView(
            type=extra_field.type,
            name=extra_field.name,
            title=extra_field.title,
            # Etc...
            data=extra_field.data,
        ) for extra_field in extra_fields
    ]