JSON and JSONB fields from a Piccolo Table is being serialized as strings in PiccoloCRUD

[hoosnick]

FastAPIWrapper(
    root_url="/courses/", 
    fastapi_app=app,
    piccolo_crud=PiccoloCRUD(
        table=Course, 
        read_only=False, 
        schema_extra={
            "deserialize_json": True
        }
    ),
    fastapi_kwargs=FastAPIKwargs(
        all_routes={"tags": ["courses"]},
    ),
)

How can i deserialize_json=True in create_pydantic_model in PiccoloCRUD?
There is a schema_extra parameter in the sources, but it does not work :(

[sinisaos]

@hoosnick This is currently not possible, but perhaps we can add deserialize_json (or some other name) to the PiccoloCRUD constructor and then pass that argument to the _pydantic_model_output and pydantic_model_plural methods. Something like this

class PiccoloCRUD(Router):
    """
    Wraps a Piccolo table with CRUD methods for use in a REST API.
    """

    max_page_size: int = 1000

    def __init__(
        ...
        deserialize_json: bool = False, # or some other name
    ) -> None:
         self.deserialize_json = deserialize_json

         def _pydantic_model_output(
             self,
             include_readable: bool = False,
             include_columns: t.Tuple[Column, ...] = (),
             nested: t.Union[bool, t.Tuple[Column, ...]] = False,
         ) -> t.Type[pydantic.BaseModel]:
            return create_pydantic_model(
                ...
                deserialize_json=self.deserialize_json,
            )

        def pydantic_model_plural(
            self,
            include_readable=False,
            include_columns: t.Tuple[Column, ...] = (),
            nested: t.Union[bool, t.Tuple[Column, ...]] = False,
        ):
            base_model: t.Any = create_pydantic_model(
                ...
                deserialize_json=self.deserialize_json,
            )
            return pydantic.create_model(
                str(self.table.__name__) + "Plural",
                __config__=Config,
                rows=(t.List[base_model], None),
            )

Usage then will be

FastAPIWrapper(
    root_url="/courses/", 
    fastapi_app=app,
    piccolo_crud=PiccoloCRUD(
        table=Course, 
        read_only=False, 
        deserialize_json=True, 
    ),
    fastapi_kwargs=FastAPIKwargs(
        all_routes={"tags": ["courses"]},
    ),
)

@dantownsend What do you think about that?

[hoosnick]

@sinisaos Thanks for the reply, for now would it be possible for me to create a separate router and create a separate pydantic model for the post/get methods?

[sinisaos]

@hoosnick Yes of course. You can create two separate models and routes for it, like in the piccolo asgi template. In out model you can pass deserialize_json=True like this

CourseModelIn: t.Any = create_pydantic_model(
    table=Course,
    model_name='CourseModelIn'
)
CourseModelOut: t.Any = create_pydantic_model(
    table=Course,
    include_default_columns=True,
    deserialize_json=True,
    model_name='CourseModelOut'
)

Hope that helps.