Skip to content
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

[feat] data fetching 리뷰 상세 / 작성 #67

Merged
merged 2 commits into from
Jan 10, 2023
Merged

Conversation

choipureum
Copy link
Member

@choipureum choipureum commented Jan 5, 2023

💡 개요

related issue: #41
test : https://deploy-preview-67--exitnow.netlify.app/review/detail/1 (현재 서버에러로 테스트불가)

📑 작업 사항

  • Like 컴포넌트 : 카운트 포지션 설정 기능 추가(마이페이지도 함께 사용)
  • Menu stateEvent 추가 및 로직 변경 & 기능 확인
  • review/detail/{reviewId} : 리뷰 상세 페칭 및 viewport 기반 무한 스크롤 구현 완료
    (useGetThemeReviewListWithInfinite.ts 참고 , useIntersectionObserver.ts 쓰시면 됩니다.)
  • like 기능 추가 (현재 서버에러로 테스트불가)
  • edit 이동 및 글 작성 완료

논의 사항

  1. 리뷰 작성시 완료이후 이동 + 토스트 등 추가 논의 => 어케할지모르겟음
  2. 리뷰 작성 Chips에 1개만 들어가도록 되어있음. 2개로 추가 확장 해야됨...
  3. 에러바운더리 추가

@choipureum choipureum self-assigned this Jan 5, 2023
@netlify
Copy link

netlify bot commented Jan 5, 2023

Deploy Preview for exitnow ready!

Name Link
🔨 Latest commit 146957e
🔍 Latest deploy log https://app.netlify.com/sites/exitnow/deploys/63b99503121eae0008b807c7
😎 Deploy Preview https://deploy-preview-67--exitnow.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@cypress
Copy link

cypress bot commented Jan 5, 2023



Test summary

9 0 0 0


Run details

Project DDD8-EXIT
Status Passed
Commit fc5624e ℹ️
Started Jan 7, 2023 3:52 PM
Ended Jan 7, 2023 3:53 PM
Duration 00:34 💡
OS Linux Ubuntu -
Browser Chrome 108

View run in Cypress Dashboard ➡️


This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

@choipureum choipureum marked this pull request as ready for review January 7, 2023 15:51
@choipureum choipureum changed the title [feat] data fetching 리뷰 상세 [feat] data fetching 리뷰 상세 / 작성 Jan 7, 2023
@@ -61,7 +77,7 @@ export default function UserInfo({ userInfo }: { userInfo: UserInfoType }) {
</ReviewUserInfoBlock>
</ReviewUserInfo>
</ReviewUserInfoContainer>
<ReviewLikeContainer>
<ReviewLikeContainer onClick={handleLikeClick}>
Copy link
Member Author

Choose a reason for hiding this comment

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

@lxxmnmn
인증, 리뷰에서 좋아요 버튼을 쓰게되는데요..!
좋아요 api 분기가 필요합니다. 이거도 논의하는걸로..!

const navigate = useNavigate();
return useMutation(() => postReviewCreateMutation(themeId, body), {
onSuccess: () => {
navigate(`/review/detail/${themeId}`);
Copy link
Member Author

Choose a reason for hiding this comment

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

등록 완료 성공이후 다음 로직이 불투명 합니다. 회의에 필수로 얘기해야할 것 같습니다.

@@ -116,6 +116,9 @@ function CustomSlide() {

export default function CafeList() {
const [cardList] = useState<CardListItemType[]>(dummyData);
// eslint-disable-next-line
const [_, setSort] = useState('');
Copy link
Member Author

Choose a reason for hiding this comment

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

데이터 페칭때 요고 사용하시면 됩니당

@choipureum
Copy link
Member Author

@jxx-mx @lxxmnmn
리뷰나 승인 부탁두립니다아

@choipureum choipureum merged commit e31f303 into main Jan 10, 2023
@choipureum choipureum deleted the feat/dataQuery branch January 10, 2023 00:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants