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

Create user and review model in backend #1

Open
ByungHeeCha opened this issue Oct 23, 2019 · 6 comments
Open

Create user and review model in backend #1

ByungHeeCha opened this issue Oct 23, 2019 · 6 comments
Assignees
Labels
enhancement New feature or request question Further information is requested

Comments

@ByungHeeCha
Copy link
Contributor

Wanted behavior will be in backend/README.md. I think we need to make backend as fast as possible for frontend.


recommendation이 필요한 api/restaurant 를 제외한 나머지 모델을 지금 만드는 것이 좋다고 생각합니다. 그래야 프론트엔드를 만드는데 도움이 될것 같습니다.

@ByungHeeCha ByungHeeCha added the enhancement New feature or request label Oct 23, 2019
@ByungHeeCha ByungHeeCha self-assigned this Oct 23, 2019
@ByungHeeCha
Copy link
Contributor Author

Should we make different apps for User, Restaurant, Review model, or combine to 1 app?

@ByungHeeCha
Copy link
Contributor Author

Combining to 1 app could make coding easier, but it may be hard to divide task to other member because of conflict problem

@ByungHeeCha ByungHeeCha added the question Further information is requested label Oct 23, 2019
@tho-kn
Copy link
Contributor

tho-kn commented Oct 24, 2019

Each model will be directly referencing each other as foreign key, I think it's impossible to divide them into different apps

@ByungHeeCha
Copy link
Contributor Author

I think django can make relation between different apps. doesn't it?


앱 사이를 연결할 수 있지 않나요?

@ByungHeeCha
Copy link
Contributor Author

I think at least User and review table can be divided by differennt apps.


최소한 User와 review 테이블은 다른 앱으로 구현할수 있을거 같아요.

@ByungHeeCha
Copy link
Contributor Author

I tried it. I think @th-snu is right. I'll fix it


실험해 보니까 @th-snu 가 맞는거 같아요. 고칠게요

jyeok added a commit that referenced this issue Dec 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants