Skip to content

Update model primary key for dynamic retrieval #23

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

Merged
merged 4 commits into from
Aug 27, 2024

Conversation

DavidSche
Copy link
Contributor

删除、更新、查询操作支持非id名称的主键,主键名称从sqlalchemy 模型定义中获取

@wu-clan
Copy link
Member

wu-clan commented Aug 26, 2024

令人兴奋的,我会稍后 review 和 merge。🙏🙏🙏

@wu-clan wu-clan self-requested a review August 26, 2024 15:16
@wu-clan
Copy link
Member

wu-clan commented Aug 26, 2024

Can you update the documentation code for this?

Also, the ruff lintting and formatting needs to be executed

@wu-clan
Copy link
Member

wu-clan commented Aug 26, 2024

I can push to this PR if needed.

@wu-clan
Copy link
Member

wu-clan commented Aug 27, 2024

@DavidSche 这太酷了

非常感谢

@wu-clan wu-clan self-requested a review August 27, 2024 08:42
@wu-clan wu-clan changed the title 删除、更新、查询支持主键名称非ID,从sqlalchemy 模型定义中获取主键 Update model primary key for dynamic retrieval Aug 27, 2024
@wu-clan wu-clan merged commit 114c7bd into fastapi-practices:master Aug 27, 2024
6 checks passed
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