Skip to content

fix(sql): accept QueryableAttribute and ColumnElement in col() helper - #2056

Closed
amir-rezaei wants to merge 2 commits into
fastapi:mainfrom
amir-rezaei:fix/col-function-support-queryable-attribute-and-column-element
Closed

fix(sql): accept QueryableAttribute and ColumnElement in col() helper#2056
amir-rezaei wants to merge 2 commits into
fastapi:mainfrom
amir-rezaei:fix/col-function-support-queryable-attribute-and-column-element

Conversation

@amir-rezaei

Copy link
Copy Markdown

Description

This PR updates the col helper in sqlmodel/sql/expression.py to accept QueryableAttribute and ColumnElement instances.

Details

  • Prevents RuntimeError when passing hybrid properties, descriptors, or column elements to col().

@YuriiMotov

Copy link
Copy Markdown
Member

Thanks for your interest!

We've recently updated the contribution guidelines and PR description template, now we ask people not to open PRs unless maintainers explicitly ask to do it.

See https://tiangolo.com/open-source/contributing/ and PR description template:

Please start with a GitHub Discussion.

Once a team member asks you to open a PR, create it and link the discussion here.

Thanks for understanding!

@YuriiMotov YuriiMotov closed this Jul 28, 2026
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.

4 participants