Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

✨Add sa_foreign_key_args and sa_foreign_key_kwargs arguments to Field #894

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

earshinov
Copy link

…to let the user define additional sqlalchemy.orm.ForeignKey attributes, such as ondelete and onupdate, for foreign keys defined in a base model.

@earshinov earshinov force-pushed the fk-attrs branch 2 times, most recently from 4a5c31d to b1ecba8 Compare April 12, 2024 16:42
…...)` to let the user define additional `sqlalchemy.orm.ForeignKey` attributes, such as `ondelete` and `onupdate`, for foreign keys defined in a base model.
@@ -68,7 +68,7 @@ jobs:
run: pip install --upgrade "pydantic>=1.10.0,<2.0.0"
- name: Install Pydantic v2
if: matrix.pydantic-version == 'pydantic-v2'
run: pip install --upgrade "pydantic>=2.0.2,<3.0.0"
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This needs to be rolled back before merging

@earshinov earshinov changed the title ✨Add foreign_key_args and foreign_key_kwargs arguments to Field ✨Add sa_foreign_key_args and sa_foreign_key_kwargs arguments to Field Apr 12, 2024
@alejsdev alejsdev added the feature New feature or request label Jul 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants