-
Notifications
You must be signed in to change notification settings - Fork 4
Feat/refactor/like optimistic update/DEVING-67 #57
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Important Review skippedAuto reviews are limited to specific labels. 🏷️ Labels to auto review (1)
Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the 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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
✅ Deploy Preview for deving-together ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Codecov ReportAll modified and coverable lines are covered by tests ✅ 🚀 New features to boost your workflow:
|
CI Status Report검사 결과
✅ 모든 검사가 통과되었습니다. |
CI Status Report검사 결과
✅ 모든 검사가 통과되었습니다. |
dbswl701
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고생하셨습니다!
| const allMeetings: SearchMeeting[] = | ||
| data?.pages.flatMap((page) => page.content) || []; | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
flatMap 좋습니다! 하나 배워갑니다!
| const res = await (token ? authAPI : basicAPI).get(`/api/v1/meetings/top`, { | ||
| params: { categoryTitle }, | ||
| }); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
authAPI로 통일해주시면 될 것 같아요!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이후 다른 브랜치에서 통일하도록 하겠습니다!
|
|
||
| return ( | ||
| <DropdownMenuPrimitive.Root onOpenChange={setIsOpen}> | ||
| <DropdownMenuPrimitive.Root onOpenChange={setIsOpen} modal={false}> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
modal 프로퍼티를 설정하신 이유가 궁금합니다!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
드롭다운이 modal true가 default인데 이 설정때문에 gnb에서 프로필 클릭시 우측에 스크롤 여백이 나왔었는데,
modal 설정을 false로 해줘야지 해당 이슈가 안생겨서 추가해두었습니당
Lee-Dong-Seok
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고생하셨습니다!👍😊

📝 주요 작업 내용
📺 스크린샷
낙관적 업데이트 적용 전

낙관적 업데이트 적용 후

카드 그리드 적용

🔗 참고 사항
ex) 의논할 점, 리뷰어가 알 필요가 있는 추가 정보나 문서, 참고 링크를 포함.
💬 리뷰 요구사항
더 좋은 방향 있으면 말씀해주시면 감사하겠습니다!
📃 관련 이슈
ex) DEVING-67