Feat(client): acorns 추가 알림 tooltip window focus 될 때 뜨도록 변경#309
Feat(client): acorns 추가 알림 tooltip window focus 될 때 뜨도록 변경#309constantly-dev wants to merge 2 commits intodevelopfrom
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Walkthrough도토리(acorns) 쿼리의 캐시 및 리페치 동작을 조정했습니다. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
✅ Storybook chromatic 배포 확인: |
jjangminii
left a comment
There was a problem hiding this comment.
저도 refetchOnWindowFocus: true 이걸로 테스트 해봤을 때 왜 안되나 했었는데 invalidateQueries이걸 까먹고 있었네요,, 👍 👍
📌 Related Issues
📄 Tasks
⭐ PR Point (To Reviewer)
updateToReadStatusAPI 함수 설정에 invalidateQueries 제거하여 바로 반영 안되도록 하였습니다.useGetAcorns에refetchOnWindowFocus옵션을 줘 pinback dashboard window가 focus될 때 refetch하도록 하였습니다.usePutArticleReadStatus에refetchType: 'none', 옵션을 통해 stale 상태로 변경은 하지만 refetch는 안되도록 설정하였습니다.시나리오
window.open(article.url, '_blank')로 아티클이 새 탭/새 창에서 열립니다.updateToReadStatus가 호출되어 해당 아티클의 read status를 즉시 변경합니다.refetchType: 'none'으로 stale 상태만 표시합니다.useGetAcorns의refetchOnWindowFocus에 의해 acorns가 다시 조회됩니다.Summary by CodeRabbit
릴리스 노트