Feat: 부스 공지 삭제 API

[gitseoyeon]

Summary

• closed [FEAT] 부스 공지 삭제 기능 #267
  DELETE /booths/notices/{notice_id}

Key Changes

• controller 클래스에 공지 삭제 추가
• service 클래스에 공지 삭제 메소드 추가
• 공지 삭제시 로그인 유저가 부스 관리자인지 판별하는 메소드를 따로 만들어서 분리

Testing

• 성공 시

• 관리자가 아닌 사람이 삭제를 할 경우

To Reviewers

• 저번 리팩토링 과정 때 TODO로 주석 처리하신 부분중에 권한 확인 클래스 따로 설정 하신 다고 본 것 같은데 우선 부스 공지 쪽에서도 권한을 확인하는 로직이 중복되어 사용되는 것 같아서 메소드 분리를 해보았습니다. 괜찮으시다면 사용된 로직 부분은 이 메소드를 활용해볼까 하는데 의견 남겨주세요!
• 그 외 질문이나 의견 있으시면 리뷰 남겨주세요!

[muncool39]

확인했습니다! 수고하셨습니다. 👍

사용된 로직 부분은 이 메소드를 활용해볼까 하는데 의견 남겨주세요!

혹시 이 부분은 해당 메서드를 이 BoothNoticeService 외의 다른 서비스들에서도 권한을 확인 하는 메서드로 활용한다는 의미실까요? 제가 작성했던 권한 확인 클래스 분리는 모든 서비스에서 사용되도록 구상한 내용이라 같은 의미인지 궁금해서 질문드려요! 만약 그렇게 된다면 매개변수 부분을 Booth로 변경해 사용하면 좋을 것 같네요 😃

[gitseoyeon]

아 제가 이해했던건 다른 서비스에서도 사용하는 느낌은 아니었고 그냥 클래스에 메소드를 따로 분리만 해두는 느낌으로 말씀드렸던건데 그냥 이 클래스에서 notice 객체를 가지고 확인하는 것도 좋아보이네요!

[gitseoyeon]

private 메소드 위치 수정 했습니다! 의견 감사합니다 😊

[muncool39]

확인했습니다! 수고하셨습니다~
한 가지 의견도 남겼는데 반영되면 바로 머지하면 될 것 같습니다!