Skip to content

Support Annotated[..., Field()]#1610

Closed
jonashaag wants to merge 2 commits intofastapi:mainfrom
jonashaag:annotated
Closed

Support Annotated[..., Field()]#1610
jonashaag wants to merge 2 commits intofastapi:mainfrom
jonashaag:annotated

Conversation

@jonashaag
Copy link
Copy Markdown

sa_column in Annotated is not reflected in the table.

# Has column DateTime(timezone=True)
my_dt_working: date time = Field(sa_column=Column(DateTime(timezone=True)))

# Has column DateTime()
my_dt_not_working: Annotated[
    datetime, Field(sa_column=Column(DateTime(timezone=True)))
]

I'm happy to try to contribute a fix but I'll need some pointer where in the code changes need to be made.

@jonashaag jonashaag closed this Oct 19, 2025
@jonashaag jonashaag deleted the annotated branch October 19, 2025 16:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants