feat: #7 core/choices.py에 planner용 Enum 추가 - #8
Open
wngjs8114 wants to merge 1 commit into
Open
Conversation
Collaborator
|
누락되었던 공통 Enum 깔끔하게 반영된거 같아!! 수고했어 (이건 여담인데 현재 Enum의 DB 저장값 스타일이 내가 적은 부분은 소문자('draft', 'high')이고 지금 적어준 부분은 대문자('PLANNED', 'DONE')가 섞여있어서 기능상 문제는 없긴 하지만 나중에 프론트엔드랑 소통할 때 불편할 수도 있으니깐 하나로 통일하는게 좋을 것 같아!) |
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
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.
관련 이슈
Closes #7
작업 내용
core/choices.py에 planner 앱에서 사용하는 Enum 5개 추가
참고
PR #4(exams 모델)에서 정리된 core/choices.py에 이어붙이는 형태로 추가했습니다.
이 Enum들은 planner/models.py(PR #2)에서 사용됩니다.
테스트
리뷰 요청
@h2ew0n