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

[BE] 스터디 API 전면 수정 #84

Merged
merged 3 commits into from
Jun 30, 2022
Merged

[BE] 스터디 API 전면 수정 #84

merged 3 commits into from
Jun 30, 2022

Conversation

gandi0330
Copy link
Contributor

@gandi0330 gandi0330 commented Jun 29, 2022

📑 제목

스터디 API에 인증 추가, 리더 아이디 체크 서비스 구현

📎

✔️ 셀프 체크리스트

최소 1명 이상의 assign을 받아야만 Merge 가능합니다.

  • Warning Message가 발생하지 않았나요?
  • Coding Convention을 준수했나요?
  • Test Code를 작성하였나요?

💬 작업 내용

구현 내용 및 작업 했던 내역

  • 스터디 테이블에 스터디 등록일 추가로 인해 dto 및 xml 변경
  • access 토큰에서 유저 id를 받아 스터디 leader id와 비교하는 leaderCheck 서비스 생성
  • 스터디 수정과 삭제 로직에 accesstoken 검증 및 leader Check 추가

🚧 PR 특이 사항

PR을 볼 때 주의깊게 봐야하거나 말하고 싶은 점

  • 스터디쪽은 서비스의 결과값에 관련해서 아무것도 안하도록 설계했습니다.
    (삽입, 수정, 삭제에서 아무런 영향을 주지 못해 0을 리턴하는 것과 잘못된 요청 값으로 인해 0을 리턴하는 것을 구분할 수 없기 때문에)

🕰 실제 소요 시간

작업을 시작하기 부터 PR을 올리기 까지 소요된 시간입니다.

  • (3시간)

@gandi0330 gandi0330 added this to the API개발 milestone Jun 29, 2022
@gandi0330 gandi0330 requested a review from lion1735 June 29, 2022 13:09
@lion1735 lion1735 merged commit 43cbcae into back Jun 30, 2022
@lion1735 lion1735 deleted the F03_BE_정현명 branch June 30, 2022 05:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants