B0X12
commented
3 months ago
Merge branch 'develop' of https://github.com/foodage/server into develop 이라는 커밋을 없애서 다시 요청해주세요. 트리가 위 사진처럼 복잡해지게 됩니다.
그리고 SecurityConfig에서 "/review"로 시작하는 모든 api의 접근을 허용하셨는데, 이렇게 되면 이 엔드포인트를 사용하는 다른 api에도 영향이 가게 됩니다. 현재는 이 엔드포인트로 시작하는 다른 api가 없지만 차후 추가될 경우 문제가 될 수 있겠죠. 이런 점을 감안했을때 현재 review 테스트 api가 하나인데 "/review/"를 쓸 이유는 별달리 없어보입니다. 해당 api에 대한 권한만 풀어주거나, 아니면 차후 영향이 갈 수 있다는 주석을 추가해주는게** 좋을 것 같습니다.
api 엔드포인트도, @PostMapping("/review/add") POST 메소드가 이미 '추가, 쓰기'에 대한 작업을 함축하여 나타내기 때문에 "작성한다"라는 행위를 엔드포인트에서 또 한 번 명시해줄 필요는 없어보입니다.
위 내용 중 수정 가능한 부분은 수정하면 좋을 것 같은데 반영후에 다시 pr 부탁드립니다.
🔖 관련 이슈
💡 작업 사항
추후 리뷰 작성 기획이 완성되면 기획에 맞게 리뷰 작성 api를 수정할 계획이며 리뷰 수정과 삭제 api도 추가할 예정입니다.