refactor: 좋아요 에러 개선 및 공유하기 기능 추가 #52
Conversation
|
@czmcm5 is attempting to deploy a commit to the tkyoun0421's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
| navigator.clipboard | ||
| .writeText(window.location.href) | ||
| .then(() => alert("URL이 복사되었습니다.")) | ||
| .catch(() => alert("복사 실패")); |
There was a problem hiding this comment.
좋은 웹 API네요 또 하나 배워갑니다 :)
감사합니다 🙇
| if (!isLike) { | ||
| setIsLike(true); | ||
| updateLike(projectID); | ||
| } else { | ||
| setIsLike(false); | ||
| updateUnLike(projectID); | ||
| } |
There was a problem hiding this comment.
비슷한 로직인 것 같아서 하나의 함수에서 관리하는 게 좋을 것 같다고 생각이 들어서 고민해보다가
근데 또 그렇게 생각해보니 복잡도가 좀 더 올라가는 부분이 있는 것 같아서..
어렵네요 개발 😢
There was a problem hiding this comment.
그쵸 ㅠㅠ api 부터 합칠까 고민했는데
넣고 빼는 행위 따라서 구분하는게 추후에 알아보기 더 쉬울거같아서 결국 나누었습니다😇
| const { mutate: updateUnLike, isPending: unLikePending } = useProjectUnLike(); | ||
|
|
||
| const initLike = user && likedUsers.includes(user.uid); | ||
| const [isLike, setIsLike] = useState(initLike || false); |
There was a problem hiding this comment.
현재 isLike 상태를 useState로만 관리하고 있는데,
유저 프로필에서 프로젝트 리스트 삭제 기능을 추가하면서 관심/좋아요 리스트를 전역상태 관리로 바꿨는데
프로필에서 관심 프로젝트를 삭제하면 디테일 페이지의 UI와 상태가 불일치 할 수도? 있을 것 같아요🥲
There was a problem hiding this comment.
관심/좋아요 리스트를 전역상태로 바꿧다는 말은 api 호출로 db값을 수정하지않는다는 말씀이신가요 ?
상세페이지의 좋아요 여부 체크는 게시글(projects 컬렉션)에 등록된 좋아요한 유저 목록을 api로 불러와 현재 로그인 한 사용자의 id와 비교하여 보여주므로 프로필의 전역상태와는 무관할 것 같습니다 😀
There was a problem hiding this comment.
엇 아니요 ! 전역 상태만 바꾸는 게 아니라, 반드시 API로 DB(파이어베이스) 값을 먼저 수정하고,
그 결과를 전역 상태에 반영해서 UI를 동기화한다는 말이었어요 !
상세페이지와 프로필 등 여러 곳에서 관심/좋아요 상태가 항상 일치하려면,
DB와 전역 상태를 항상 동기화하는 구조가 안전하지 않을까 생각합니다 !
There was a problem hiding this comment.
아하 네~
상세 페이지에서는 항상 최신 상태의 db값을 들고오며 user 프로필에서 uid 값 이외엔 사용하지 않으니 수정하지 않아도 괜찮을 것 같습니다 😉
tkyoun0421
merged commit bb1419c
into
amicable-development-center:develop
3 participants
개요
비로그인 시 좋아요 버튼을 누르면 좋아요가 등록되지 않아도 활성화 되는 에러를 수정하였습니다. 적고보니까 fix가 맞는걸지도..
공유하기 버튼을 누르면 현재 url이 복사 되도록 하였습니다.
변경 사항
구현 내용
개발 후기 및 개선사항
이번 작업에서 배운 점
어려웠던 점 / 에로사항
다음에 개선하고 싶은 점
팀원들과 공유하고 싶은 팁