Refactor foreign keys and relationships to pure logic #901
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Member
Author
|
TODO:
|
# Conflicts: # backend/app/admin/crud/crud_dept.py # backend/app/admin/crud/crud_user.py
Collaborator
model这样设计，crud可正常使用selectinload 可以验证一下这个可行性，可能会更加优雅 |
Member
Author
|
此 PR 是为了删除任何表之间的物理外键约束（fk、cascade) 定义 事实上，仅使用 relationship 而不使用 fk 也可以正常工作，只是需要手动处理级联操作（cascade） 综上所述，仅使用 relationship 似乎也能达到此 PR 目标，但是，对于明确规定禁止使用 relationship 的要求，这仍然无法满足 我的最终目标是，在此 PR 被合并之前，完成 PR 中列出的所有任务，然后将主分支进行最后发版并创建备份分支 |
Member
Author
|
就像我在其他地方回复的一样，相比于以下实现，我已为目前的封装实现做了很大努力 
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Close #825