-
Notifications
You must be signed in to change notification settings - Fork 4
2020.11.09 개발 일지
rockpell edited this page Nov 9, 2020
·
3 revisions
11월 9일
- deleteComment, editComment, getComments API를 완성하였습니다.
- 기존에 적용된 createComment API가 최신화가 안되어 있어 수정이 필요합니다.
- Oauth
- 공부/작성중
- 단순히 로그인 대채용인줄 알았습니다
- 인증 과정
- 서버에 github Oauth url 받아옵니다
- 프론트에서 받아온 url을 타고 인증을 해서 authentication code를 받습니다
- code를 서버에 보냅니다.
- 서버는 code와 client_id, client secret를 보내서 토큰을 받아서 해당 유저의 정보를 받아옵니다.
- 받아온 정보를 토대로 유저 식별을 한다. 예를 들어서 토큰을 만들어서 전달합니다.
- 공부/작성중
- axios를 사용해 label 데이터를 가져왔습니다.
- LabelFileterButton, LabelSelector 부분에 적용했습니다.
- webpack watch를 사용했기 때문에 cors 문제는 안났었습니다.
- useState와 useEffect를 사용했습니다. 후에 contextAPI 사용시 수정해야 합니다.
- 개인적으로 api 호출하는 건 따로 파일로 해놓는 게 좋을 것 같다는 생각이 듭니다. 일단은 개별적으로 해놓고 만나서 고치면 될 것 같습니다.
- Edit Issue API
- 이슈 수정 API를 구현했습니다.
- transaction을 사용해서 Issue에 대한 정보뿐만 아니라 Label과 Assignee 정보까지 모두 update에 성공했을 때 COMMIT하고 실패했다면 ROLLBACK 하도록 했습니다.
- validate 함수 부분은 수정이 필요합니다.(함수로 따로 분리)
- CORS Middleware 추가(backend)
- package.json에 cors dependency를 추가했습니다.
- backend의 app.js에 cors 설정을 추가했습니다. (cors middleware 적용)
- 이슈 목록 페이지에서 API를 호출해 이슈 목록을 가져오는 작업중입니다.
- 반환하는 error message가 대소문자등 미세하게 차이가 있어 공동작업을 할 때 변경하기로 하였습니다.
- inputs missing
- Bad Request
- axios 사용시 async, await과 then, catch 사용 방법에 대한 논의를 하였습니다.
- 프론트 페이지에서 axios 오류가 발생하면 alert로 경고를 띄우기로 정하였습니다.
- api를 요청하는 파일을 위한 디렉토리를 만들기로 하였습니다.(/apis)
개발 일지
데일리 스크럼