-
Notifications
You must be signed in to change notification settings - Fork 0
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
Feat : 스터디룸 삭제 API #14
The head ref may contain hidden characters: "LINKODE-16-BE-API-\uC2A4\uD130\uB514\uB8F8-\uC0AD\uC81C-API"
Conversation
추후 회원 기능 구현 완료되면 바뀔사항 많아서 대략 틀만 작성
일대다 관계여야하는데 오류가있어서 수정했습니다.
테이블 설계시 색상 속성이 누락된것 같아 추가했습니다.
gitignore 에 application-local.yml 추가
LoginController 에 있던 getAccessToken,getUserInfo 메소드 서비스로 옮김
isMember 로 하니까 인식을 잘못하는 오류발생
아직 jwt 반환은 미완성
아직 jwt 반환은 미완성
삭제요청한 스터디룸이 없는경우 500이 아닌 예외처리 메시지 반환하도록
기존에 githubId 로 토큰을 만들었는데 memberId 로 만들도록 수정
헤더로 들어온 토큰이 유효해야지만 해당 api 이용할 수 있도록 security 수정
JwtProvider jwtProvider; | ||
|
||
@PatchMapping("/removal") | ||
public BaseResponse<BaseExceptionResponseStatus> deleteStudyroom(@RequestHeader("Authorization") String authorization, @RequestHeader @RequestParam long studyroomId){ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
스터디룸id 는 리퀘스트 파람으로 받는거 맞나요?? @RequestHeader 도 같이 사용되어야하는건지 확인 부탁드립니다..!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
모두 맞는내용입니다!
오타가있습니다.
먼저 데이터베이스 무결성을 위해서 스터디룸만 delete 를 하고 멤버_스터디룸 은 active 로 데이터를 살려두는 것은 문제가 있어보입니다. 이 부분 고민이 필요할 것 같습니다. |
삭제시에 멤버_스터디룸또한 변경하도록 하였습니다! |
별거는 아닌데 쿼리문에 nickname 이랑 color 순서가 바뀌게 작성되어 있어서 테이블에 바뀐 상태로 들어갑니다! 지금 pr 에서 문제는 아니지만 눈에 보여서.. |
…스터디룸-삭제-API # Conflicts: # src/main/java/com/linkode/api_server/common/response/BaseResponse.java # src/main/java/com/linkode/api_server/controller/StudyroomController.java # src/main/java/com/linkode/api_server/repository/MemberstudyroomRepository.java # src/main/java/com/linkode/api_server/repository/StudyroomRepository.java # src/main/java/com/linkode/api_server/service/StudyroomService.java
앗 확인했습니다! 감사합니다! |
…nto LINKODE-16-BE-API-스터디룸-삭제-API
요약 (Summary)
DELETE
로 변경한 뒤 맴버스터디룸테이블에서도 해당 스터디룸의 관계를 모두DELETE
하려했으나, 스터디룸 자체가DELETE
되버리면 스터디룸이 조회되지않을것이고Soft Delete
의 목적이 데이터 복구의 안정성이라는 측면에서스터디룸 테이블
의 상태만 변경하고 관계는 유지해 두는것이 유리하다고 판단했습니다.🔑 변경 사항 (Key Changes)
📝 리뷰 요구사항 (To Reviewers)
확인 방법
.yml을 자신의 로컬에맞게 변경하세요
쿼리문
소셜로그인 주소
API 요청 주소 [PATCH}
studyroomId=2 에서는 방장이 아니라 삭제되지 않아야합니다.
요청 실패 로그 메세지
포스트맨 예시
워크벤치에서 상태가 변한 모습