Closed openmpy closed 3 months ago
@jjangsky 님~ 피드백 주신 부분 수정 해봤습니다.
한번 확인 부탁드리겠습니다. 감사합니다 👍
@openmpy 님 커밋 내역 확인 완료하였습니다~
한 가지 희망사항이 있다면 외부API Enum Class에 관리용으로 get인지 post요청인지 구분할 수 있으면 더 좋을 것 같습니다.
요청방식이 따로 사용하지는 않더라도 나중에 보기 편하고 혼동이 없을것 같아 제안드립니다~ 아니면 주석으로 관리해도 문제 없을것 같습니다 !
PR Merge 진행 하도록 하겠습니다.
관련 이슈
18
변경 사항
체크 목록
리뷰 요청
@jjangsky 님 GitHub 회원가입 기능만 일단 구현한 상태입니다. 빠른 시일내로 로그인 기능까지 구현해보도록 하겠습니다. OAuth 관련해서는 위에 내용을 읽어보시고 @jjangsky 님이 따로 설정을 해주셔야 할거 같습니다. 확인하시고 편하게 코멘트 부탁드리겠습니다. 감사합니다.
참고 자료
https://docs.github.com/ko/apps/oauth-apps/building-oauth-apps/authorizing-oauth-apps