search

depromeet / reasonable

1 stars 1 forks source link

Added University List feature #1

Closed wotomas closed 7 years ago

wotomas commented 7 years ago

GET /university/list/

만들고싶었는데, 출근하기 전에 시간도 좀 부족했어서 많이는 못봤땅 로컬에서 실행시켜볼려고 python manage.py runserver 도 돌렸는뎅 ㅠㅠ 내꺼에선 뭐 rest 뭐시기 못찾는다고 에러뜨넹

리뷰 해주구 찬찬히 설명해주세용 ㅎㅎㅎ

JeongtaekLim commented 7 years ago

아아, Django project dependency를 pip으로 설치해줘야되 https://github.com/JeongTaekLim/TIL/blob/master/Django/Pip.md 에서처럼, 

env2/bin/pip install -r requirements.txt

해주면 되 물론 python virtual env 설정해준 이후에!

JeongtaekLim commented 7 years ago

Serializers.py에 UniversityListSerializer의 경우, 지금 보면 modelSerializer를 상속하는 것을 볼 수 있을거야 def같은 경우에, 해당 model(그러니까 지금같은 경우 university.models의 University겟지?)에 모델 속성 이외에 추가적인 필드를 커스텀하게 만들어 넣고 싶을때 쓰는거라, 그냥 모델의 필드들만 필요하면 굳이 안써도되

JeongtaekLim commented 7 years ago

한번 로컬에서 돌려보고 이상없으면 APIDocs.md에도 추가를 부탁드려요 모르는거 있음 또 물어봐줘!