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

支持使用组合定义模型 #70

Closed
flycash opened this issue May 23, 2022 · 11 comments
Closed

支持使用组合定义模型 #70

flycash opened this issue May 23, 2022 · 11 comments
Milestone

Comments

@flycash
Copy link
Contributor

flycash commented May 23, 2022

设计文档

@flycash flycash added this to the 0.0.1 - CURD milestone May 25, 2022
@github-actions
Copy link

This issue is inactive for a long time.

@flyhigher139
Copy link
Collaborator

@flycash 老师,这个issue我认领过来做吧

@flycash
Copy link
Contributor Author

flycash commented Aug 25, 2022

ok,你需要可以分成好几个步骤来:

  • 修改 model 部分,要计算出来组合形态下的 offset 和 indexes
  • 修改 valuer 部分,这个部分原则上不用修改,只需要加一些测试进去
  • 修改 CRUD 的几个,原则上也不需要修改,只需要加一些测试进去

@flyhigher139
Copy link
Collaborator

收到~

@flycash
Copy link
Contributor Author

flycash commented Nov 8, 2022

@flyhigher139 啥时候把测试补充一下?

@flyhigher139
Copy link
Collaborator

@flycash 老师,我周末前出一稿吧,这几天有些杂事,晚上没搞代码

@flycash
Copy link
Contributor Author

flycash commented Nov 10, 2022

👌🏻

@flyhigher139
Copy link
Collaborator

@flyhigher139 啥时候把测试补充一下?

老师,测试用例已补充: #104

@flycash
Copy link
Contributor Author

flycash commented Nov 15, 2022

最后还有一个步骤,加上简单的增删改查集成测试

@flyhigher139
Copy link
Collaborator

最后还有一个步骤,加上简单的增删改查集成测试

好的~

@flyhigher139
Copy link
Collaborator

@flycash 集成测试已经添加

@flycash flycash closed this as completed Dec 6, 2022
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

No branches or pull requests

2 participants