search

CAPU-ENG / CAPUHome-API

Core system for the online platform of CAPU.
MIT License
2 stars 3 forks source link

New models. #17

Closed JeldorPKU closed 8 years ago

JeldorPKU commented 9 years ago

New models built according to #6 .

JeldorPKU commented 8 years ago

@huxuan I add two foreign keys and relationships to the model in commit 659e1bc. Please check whether I am doing it the right way.

huxuan commented 8 years ago

这个PR别拖太久了吧，测试一下新建表木有大问题，把字段名称啥的再斟酌一下就准备ready for merge吧。

JeldorPKU commented 8 years ago

@huxuan 调整了一下顺序和名称，然后uid那些如果没有歧义就直接uid了，涉及多个用户的再按角色注明。我测试一下～

JeldorPKU commented 8 years ago

@huxuan Basic database schema completed. Ready for merge. Please check!

huxuan commented 8 years ago

字段的问题，开发过程当中再慢慢改吧，这个pr暂时没啥问题了，我就先merge了。

huxuan commented 8 years ago

已经merge的PR就不要再挖细节的坑了，挖了也干不了啥（难道要revert整个PR？） 直接交新的PR或者issue吧。