[Fix] : 준비됐지 임전윤채 #848 #850
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
물론이지 호주정우
|
🎉 @sinryuji 님 랜덤 리뷰어로 당첨되셨습니다! 리뷰를 부탁드립니다. 🙏 |
|
편집과 같은 작성자 권한으로 실행 |
유저, 삭제 메서드 호출 관련 예외 처리
편집처럼 삭제도 작성자 확인로직 추가.
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
물론이지 호주청정우
변경 사항
컨트롤러에서 토큰을 사용하지 않아 추가.
조건에 부합되는 ( 모집 완료가 아닌, 팀 승인을 받은 멤버가 자신 제외 존재하지 않는 경우 )
그가 속한 해당 팀에 관련된 데이터베이스 연관 테이블 또한 삭제하도록 수정.
( 팀-유저, 팀-역할 ), 그 뒤로 모집글을 삭제.
지금은 작성자만 권한 부여.
편집 요청과 같이 AuthorCheck 사용
대신 작성자가 아닌 회원이 삭제 요청할 때
게시글 작성자만 수정할 수 있습니다. 라는 메시지 반환
테스트 결과