Motov Yurii

Results 218 comments of Motov Yurii

This seems to work well in current version (0.0.24). Initial migration is now different: ``` op.create_table( 'event', sa.Column('event_uuid', sa.Uuid(), server_default=sa.text('gen_random_uuid()'), nullable=False), sa.PrimaryKeyConstraint('event_uuid') ) ``` And I managed to create second...

So, current solution is to use `Field(sa_column=Column("metadata", ...)`. I personally think it would be good to use `alias` parameter as a default value for column name. I just created an...

This is not SQLModel issue. Please refer to the documentation of the driver you are using and follow installation instructions: https://docs.sqlalchemy.org/en/20/dialects/mysql.html

PR was closed due to inactivity for 30 days.. @KimigaiiWuyi, would you like to continue working on this? I can re-open it

Yes, I think we need to add some tests. And, we need to resolve this: https://github.com/fastapi/sqlmodel/pull/1035#issuecomment-3216354850 > Indeed, you are correct. Although this situation is rare, it must be taken...

I think we should change: ```py if current_schema_extra: for key, value in current_schema_extra.items(): if key in FIELD_ACCEPTED_KWARGS: current_pydantic_kwargs[key] = value else: current_json_schema_extra[key] = value ``` to ```py if IS_PYDANTIC_V2: for...

We just discussed the idea of adding `pydantic_kwargs` parameter with Sebastian and he thinks we don't need to add it (we will later support passing all Pydantic-related parameters directly). So,...

@KimigaiiWuyi, I suggested my way of fixing this, please review: * Removed `pydantic_kwargs` parameter, updated texts that mentioned it * Renamed `pydantic_kwargs` local var into `field_info_kwargs` as it's used in...

> Thanks for approval! Who is responsible for merging this, what are the rules in this repo? Only [Sebastian](https://github.com/tiangolo) can merge it. I already forwarded it to him. We should...

> Relevant part of the docstrings: > > [fastapi/fastapi/applications.py](https://github.com/fastapi/fastapi/blob/9606b916ef7c883f5ebeac1bf3db9adf5ae646a9/fastapi/applications.py#L299-L300) > > Lines 299 to 300 in [9606b91](/fastapi/fastapi/commit/9606b916ef7c883f5ebeac1bf3db9adf5ae646a9) > > If the servers `list` is not provided, or is an empty...