Feat: 가이드라인 삭제 기능 추가

[eunsoni]

#️⃣ 연관된 이슈

• Feat: 가이드라인 삭제 api 추가 구현 및 가이드라인 dto 수정 #107

📝 작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요.(이미지 첨부 가능)

• 가이드라인 서비스단에 삭제 메서드 추가
• 가이드라인 컨트롤러에 삭제 api 추가
• 가이드라인 엔터티 id 속성 이름 Id로 수정
• 사용하지 않는 exception 삭제
• Swagger 사용해서 삭제 api 테스트 (정상 작동)

스크린샷 (선택)

💬 리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

API 설계할 때 삭제 api이 빠져서 URL을 제가 정해봤습니다 수정원하시면 말씀해주세요!!

⏰ 현재 버그

✏ Git Close

close #107

[GitJIHO]

변경사항 확인했습니다~~

[GitJIHO]

카멜케이스 표기에서 다른걸로 바꾸신 이유가 있나요?

[GitJIHO]

변경코드에는 없는데

public record GuardGuidelineResponse(
        Long Id,
        GuardGuideline.Type type,
        String title,
        String content) {
}

가이드라인 response DTO에도 카멜케이스가 아닌 다른 표기법으로 id가 쓰여있는 것 같아요~

[eunsoni]

response에 Id -> id 로 바꿔달라는 프론트 요청이었는데 제가 잘못이해했었네요...!! ㅜ
감사합니다 둘 다 id로 다시 바꿔둘게요!

[zzoe2346]

삭제 기능 추가 고생하셨습니다~!

[zzoe2346]

Suggested change

	@GetMapping("/delete")
	public ResponseEntity<String> deleteGuardGuideline(@MemberId Long memberId, @RequestParam("guidelineId") Long guidelineId) {
	guardGuidelineService.deleteGuardGuideline(memberId, guidelineId);
	return ResponseEntity.ok("가이드라인이 삭제되었습니다.");
	}
	@DeleteMapping("/{guidelineId}")
	public ResponseEntity<String> deleteGuardGuideline(@MemberId Long memberId, @PathVariable("guidelineId") Long guidelineId) {
	guardGuidelineService.deleteGuardGuideline(memberId, guidelineId);
	return ResponseEntity.ok("가이드라인이 삭제되었습니다.");
	}

좀 더 RESTful 한 느낌이 어떨까 싶어요!~ 한번 제안드려봅니다 😀

[GitJIHO]

엇 리뷰에서 이걸 놓쳤네요
성훈님 의견에 동의합니다

[eunsoni]

앗 이 부분 놓쳤네요
delete 사용하는 것으로 수정해두겠습니다:>
혹시 query parameter를 path variable로 바꾸는 것을 제안하시는 이유를 알 수 있을까요?
저는 주소창에서 url을 변경해서 다른 가이드라인을 지우는 것이 적절하지 않을 것 같다고 생각해서 일단은 query parameter로 작성했었습니다

[zzoe2346]

은선님 말씀도 맞다고 생각합니다! 저는 큰 의미는 없었고, 저희의 다른 Delete API 들이 path variable로 URL을 만들어 놓은것을 보고 저렇게 제안드려봤어요. 은선님 도메인이시니 뭐든 선택해도 좋으실거 같습니다.

[eunsoni]

넵 수정해두었습니다~!

[2iedo]

고생하셨어요~~!

[GitJIHO]

수정사항 확인했습니다~ 👍