🚀 3단계 - 즐겨찾기 기능 구현 #248

mintheon · 2022-03-06T12:39:14Z

마지막 기능 구현은 생각보다 어렵지 않았네요..! 많이 늦었지만 마지막까지 달려가겠습니다.

요구사항

즐겨 찾기 기능을 구현하기
로그인이 필요한 API 요청 시 유효하지 않은 경우 401 응답 내려주기

- 즐겨찾기 등록 기능 추가 - 기존 인수테스트에서 token을 보내지 않던 부분 수정 - 즐겨찾기 생성 메서드 분리

- 즐겨찾기 조회 기능 추가 - 다른 테스트 코드에 token 넣을 수 있도록 auth 추가 - 인수 테스트 여러 경우 확인 위해 조회시 즐겨찾기 두개 등록하도록 변경

parkeeseul

리뷰가 많이 늦었네요! 🙏
몇 가지 소소한 코멘트 남겨두었으니 확인 부탁드리겠습니다 😄
즐거운 금요일 되세요!

parkeeseul · 2022-03-10T13:49:05Z

src/main/java/nextstep/favorite/ui/FavoriteController.java

+	@GetMapping("/favorites")
+	public ResponseEntity<List<FavoriteResponse>> getFavorites(@AuthenticationPrincipal LoginMember loginMember) {
+		List<FavoriteResponse> favorites = favoriteService.findAllFavorite();
+
+		return ResponseEntity.ok().body(favorites);
+	}


로그인된 사용자의 즐겨찾기 뿐만 아니라 모든 사용자의 즐겨찾기가 조회 될 것 같아요! 🤔

parkeeseul · 2022-03-10T14:09:41Z

src/main/java/nextstep/favorite/application/FavoriteService.java

+	public void deleteFavorite(Long MemberId, Long favoriteId) {
+		favoriteRepository.deleteById(favoriteId);
+	}


memberId 를 사용하지 않고 있네요
로그인한 사용자가 아닌 다른 사람들의 즐겨찾기도 삭제 될 수 있겠어요 😢

parkeeseul · 2022-03-10T14:23:09Z

src/main/java/nextstep/favorite/application/FavoriteService.java

+	@Transactional(readOnly = true)
+	public List<FavoriteResponse> findAllFavorite() {


조회 부분 외에 다른 기능에는 Transaction 이 필요 없을까요? 😄

mintheon added 7 commits March 1, 2022 02:28

refactor: member 관련 테스트 패키지 변경

958c5b4

test: 즐겨찾기 기능 인수테스트 추가

667daba

feat: 즐겨찾기 등록 기능 추가

d37c54a

- 즐겨찾기 등록 기능 추가 - 기존 인수테스트에서 token을 보내지 않던 부분 수정 - 즐겨찾기 생성 메서드 분리

feat: 즐겨찾기 조회 기능 추가

e40c19f

- 즐겨찾기 조회 기능 추가 - 다른 테스트 코드에 token 넣을 수 있도록 auth 추가 - 인수 테스트 여러 경우 확인 위해 조회시 즐겨찾기 두개 등록하도록 변경

feat: 즐겨찾기 삭제 기능 추가

56ea6c0

refactor: 리스트 요청 메서드 이름 변경 & 트랜잭션 readonly 처리

75e8097

fix: 비로그인 상태에서 내정보, 즐겨찾기 401 예외 처리

cc66c9b

parkeeseul requested changes Mar 10, 2022

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

🚀 3단계 - 즐겨찾기 기능 구현 #248

🚀 3단계 - 즐겨찾기 기능 구현 #248

mintheon commented Mar 6, 2022 •

edited

Loading

parkeeseul left a comment

parkeeseul Mar 10, 2022

parkeeseul Mar 10, 2022

parkeeseul Mar 10, 2022

		@Transactional(readOnly = true)
		public List<FavoriteResponse> findAllFavorite() {

🚀 3단계 - 즐겨찾기 기능 구현 #248

Are you sure you want to change the base?

🚀 3단계 - 즐겨찾기 기능 구현 #248

Conversation

mintheon commented Mar 6, 2022 • edited Loading

요구사항

parkeeseul left a comment

Choose a reason for hiding this comment

parkeeseul Mar 10, 2022

Choose a reason for hiding this comment

parkeeseul Mar 10, 2022

Choose a reason for hiding this comment

parkeeseul Mar 10, 2022

Choose a reason for hiding this comment

mintheon commented Mar 6, 2022 •

edited

Loading