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

4 주차 PR #6

Open
wants to merge 17 commits into
base: master
Choose a base branch
from
Open

4 주차 PR #6

wants to merge 17 commits into from

Conversation

nette74
Copy link
Collaborator

@nette74 nette74 commented Nov 9, 2022

[접근 방법]

API를 REST 방식으로 바꾸는 과정을 수행하였습니다. 팀프로젝트에서 REST 통신만을 사용하였기 때문에, 한편으론 익숙한 부분이었으나, 로직 구현 부분에서 ERD 에 관한 이해가 부족했는지 구현을 못 한 부분이 있습니다.
또한 진행했던 팀프로젝트와 과제의 컨트롤러 return 구조가 달라서 헷갈리는 부분이 있습니다.
doc 과 jwt 는 시간이 부족하여 구현하지 못하였습니다.

[진행 현황]

  • JWT 로그인 구현(POST /api/v1/member/login)
  • 내 도서 리스트 구현(GET /api/v1/myBooks)
  • 내 도서 상세정보 구현(GET /api/v1/myBooks/{id})
  • 로그인 한 회원의 정보 구현(GET /api/v1/member/me)
  • Spring Doc 으로 API 문서화(크롬 /swagger-ui/index.html )
  • 엑세스 토큰 화이트리스트 구현(Member 엔티티에 accessToken 필드 추가)
  • 리액트에서 작동되도록

[특이사항]
구현 과정에서 아쉬웠던 점 / 궁금했던 점을 정리합니다.

  • 도서 상세정보의 챕터에 post 가 들어가는 과정을 이해하지 못해 구현에 어려움이 있었습니다.

[Refactoring]

  • jwt 인증을 도입합니다
  • doc 을 작성합니다.

Copy link
Member

@sh970901 sh970901 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

엔티티 객체를 바로 활용하는 것 보다 DTO를 만들어 사용하는 방법 잘 봤습니다.
이번주차도 고생많으셨습니다!!

"S-1",
"회원 조회 성공",
Ut.mapOf(
"member", memberDto
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

dto를 활용해서 요구사항에 맞추는게 더 좋겠네요!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants