feat: 북마크 기능 구현 #139
feat: 북마크 기능 구현 #139
Conversation
| const { data: member } = useMemberQuery(); | ||
|
|
||
| const handleToggleFavorite = async () => { | ||
| mutate({ favorite: !favorite }); | ||
| favoriteMutate({ favorite: !favorite }); | ||
| bookmarkMutate({ bookmark: !favorite }); |
There was a problem hiding this comment.
이 부분에 대해서 물어볼 게 있는데, 사실상 북마크가 좋아요랑 똑같잖아요.
같은 버튼이고, 누르면 boolean 값이 오는게.
그래서, 따로 bookmark 만들까 하다가 그냥 저 값 재사용했는데 괜찮나요?
There was a problem hiding this comment.
그 꿀조합 좋아요 -> 북마크 기능으로 바뀐거 맞죠?? 그럼 favoriteMutate는 없어져도 되는거 아닌가요...? 헷갈리네
There was a problem hiding this comment.
어...저도 이거 좀 헷갈리는데, 일단 api는 두개더라구요?
근데 favorite count는 favoriteMutate랑 관련있는 거 아녜요?
bookmarkMutation으로도 해결 가능하나???
There was a problem hiding this comment.
음 지금 코드로 보면 count -> favoriteMutation으로 작동하고 favorite(bookmark) -> bookmarkMutation으로 작동하는데 ui상엔 문제가 없지만 중복로직이라 둘이 따로 둘 필요가 없는거 같아여! 나중에 백엔드랑 얘기해보고 리팩토링 하면 좋을 것 같아요
There was a problem hiding this comment.
오케 이해 완! 사실상 favorite냐 bookmark냐 이름만 다르긴 해....
회의 때 바꿔달라고 얘기합시다~~
|
🔗 배포된 storybook 주소: https://65f015a4aed45406385006ee-mdqzqskdym.chromatic.com/ |
Issue
✨ 구현한 기능
북마크 기능을 구하였습니다.
params를 명세에 맞게 바꿨습니다.
📢 논의하고 싶은 내용
제대로 작성했는지 확인해주세용.
특히 tanstack query 사용한 부분 key나 등등 맞게 썼는지 확인 부탁드립니다.
좋아요 버튼을 누르면, 좋아요랑 북마크 기능이 동시에 작동하거든여?
그래서 좋아요 버튼 query랑 비슷하게 썼어요.
근데 궁금한게, 좋아요에 query key를 recipeDetail로 했던데
그게 원래 v1에서는 recipeDetail에서만 좋아요 가능해서 그렇게 했던거죠??
🎸 기타
x
⏰ 일정