Skip to content

Feat : 레시피 좋아요 싫어요#44

Merged
kingkang85 merged 2 commits intodevelopfrom
20-레시피-좋아요-싫어요-기능
Dec 16, 2024

Hidden character warning

The head ref may contain hidden characters: "20-\ub808\uc2dc\ud53c-\uc88b\uc544\uc694-\uc2eb\uc5b4\uc694-\uae30\ub2a5"
Merged

Feat : 레시피 좋아요 싫어요#44
kingkang85 merged 2 commits intodevelopfrom
20-레시피-좋아요-싫어요-기능

Conversation

@uni-zun
Copy link
Copy Markdown
Contributor

@uni-zun uni-zun commented Dec 16, 2024

📋 요약

  • 각 레시피별 좋아요 싫어요를 회원들이 달수있다, 추가로 좋아요 싫어요 개수도 세는 로직 구성함.

🛠 변경 사항

  • 이번 Pull Request에서 작업한 주요 변경 사항은 다음과 같습니다:
    • 새로운 기능 추가 또는 버그 수정
    • 코드 리팩토링 또는 최적화
    • 문서 업데이트

🔗 관련 이슈

  • 이 PR로 해결되는 이슈: #[이슈 번호]
  • 관련된 이슈: #[이슈 번호]

📸 스크린샷 또는 GIF (해당되는 경우)

테스트

왼쪽부터 4가지 상태 설명

  1. 아무것도 안한 상태
  2. 1번 레시피를 1번 회원이 좋아요(true) 눌러 likeCount 가 1로 오름
  3. 1번 레시피를 3번 회원이 좋아요 눌러 likeCount가 2 로 오름
  4. 1번 레시피를 1번 회원이 싫어요(false) 눌러 likeCount 는 1 이 줄고 dislikeCount 는 1 이 오름을 확인
    ++ 추가로 likeStatus 를 null 로 하면 올라가 있는 count 는 내려가는걸 확인하였음

✅ 체크리스트

  • 코드가 정상적으로 동작하는지 테스트했습니다.
  • 관련 문서를 작성하거나 업데이트했습니다. (해당되는 경우)
  • 변경 사항을 반영한 테스트 코드를 추가했습니다.
  • 모든 테스트가 성공적으로 통과했습니다.

🛡 테스트 방법

  • 변경 사항을 확인하기 위한 테스트 방법을 단계별로 작성해주세요:
    1. 브랜치를 로컬로 가져옵니다.
    2. 아래 명령어를 사용해 애플리케이션 또는 테스트를 실행합니다:
      # 예시 명령어
      ./run_tests.sh
      
    3. 예상 동작이나 출력 결과를 확인합니다.

📚 추가 참고 사항

  • 이 PR과 관련해 리뷰어가 알아야 할 추가 내용이 있다면 작성해주세요.

1. 각 레시피별 좋아요 싫어요를 회원당 한 가지 선택 가능 ( null 도 있어서 안 누를 수 있다)
2. 좋아요 싫어요 수를 계산한값이 나온다.
Related to : #20
Copy link
Copy Markdown
Contributor

@KIM9909 KIM9909 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이 PR 좋아요 + 1

Copy link
Copy Markdown
Contributor

@MinFe4990 MinFe4990 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

!check

@yunjaeeun
Copy link
Copy Markdown
Contributor

테이블을 늘리기 보다 컬럼을 추가해서 해결하는게 인상깊네요

Copy link
Copy Markdown
Contributor

@LeeGukgeon LeeGukgeon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

현재 로그인 되어있는 userPk 정보를 SecurityContextHolder 에서 가져와서 request 에 주입하는 로직이 필요할 것 같은데 제가 한번 해봐도 될까요?

Copy link
Copy Markdown
Contributor

@kingkang85 kingkang85 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

확인했습니다!

@kingkang85 kingkang85 merged commit eb08597 into develop Dec 16, 2024
@uni-zun
Copy link
Copy Markdown
Contributor Author

uni-zun commented Dec 16, 2024

현재 로그인 되어있는 userPk 정보를 SecurityContextHolder 에서 가져와서 request 에 주입하는 로직이 필요할 것 같은데 제가 한번 해봐도 될까요?

오 넵!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants