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

🎉 리다이랙트 및 라우팅 추가 + 버그 수정 #138

Merged
merged 3 commits into from
Nov 29, 2023

Conversation

juyeon-park
Copy link
Contributor

- 목적

관련 티켓 번호: 359


- 주요 변경 사항

  • 거래성사관련 알림 클릭시 채팅방목록으로 리다이랙트로하도록 수정
  • 상세정보페이지에서 카테고리 누르면 카테고리 필터된채로 전체 목록 페이지로 이동 추가
  • 제안모달에서 이미지 누르면 상세정보 페이지로 이동하도록 추가
  • 찜 뒤로가기&되돌아오기 했을 때 업데이트 안되어있던 버그 수정
  • 채팅방 처음 접속시 로딩 되도록 추가

기타 사항 (선택)


- 스크린샷 (선택)

@juyeon-park juyeon-park self-assigned this Nov 29, 2023
<u
className="cursor-pointer"
onClick={() =>
router.push(`${AppPath.cards()}?category=${category}`)
Copy link
Contributor

Choose a reason for hiding this comment

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

utils 디렉토리에 getQueryParams 함수를 만들어놓았는데 한번 괜찮으시면 써보시는게 어떨까요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

오!!! 이런 좋은 유틸 함수가 있었군요! 바로 수정했습니다. 감사합니다

Comment on lines +56 to +70
<Suspense fallback={<Loading />}>
{['OFFER', 'POKE'].map((type) => (
<TabsContent
key={type}
value={type}
className="flex flex-col data-[state=inactive]:hidden h-[402px] overflow-y-auto pr-2"
>
{!pokeAvailable && type === 'POKE' ? (
<PokeUnavailableInfo />
) : (
filterData(type)
)}
</TabsContent>
))}
</Suspense>
Copy link
Contributor

Choose a reason for hiding this comment

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

무한스크롤이 아니라서 그런지 Suspense를 적용 하셨네요...
useInfinteQueries는 Suspense가 동작이 안된다고 하더라고요.
혹시 아시는 부분이 있을까요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

동현님이 올린 PR에 답변 작성했는데 useInfiniteQueries에 suspsense 작동이 안된다?는 들어본적이 없는 것같습니다. 한번 useSuspenseInfiniteQuery를 이용해보시겠어요? 근데 이것도 useInfiniteQuery에서 있던 suspense, enabled 등 옵션을 기본적으로 가지게 한 구조라 확인해보셔야될 것 같습니다.

@@ -11,8 +11,15 @@ type NotificationCardProps = {
const NotificationCard = ({
notification: { content, cardId, read },
}: NotificationCardProps) => {
const isCompleteRequestNotification = content.includes('성사')
Copy link
Contributor

Choose a reason for hiding this comment

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

이 부분은 나중에 백엔드랑 협의를 해봐야겠네요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

어제 QA에서 나온 내용을 토대로 거래성사와 관련된 부분은 채팅방목록으로 리다이랙트 되게 한 것인데요! 어떤 점을 백엔드랑 더 협의해 봐야할지 동현님의 질문 의도가 궁금합니다!!!! 물론 거래 성사 , 거래성사 이렇게 띄운 상태로 오는 메세지도 있어서 성사 단어만 가졌을 경우 리다이랙트 하도록 설정했습니다!

@@ -5,7 +5,7 @@ import lottieJson from '../../public/loading.json'

const Loading = () => {
return (
<div className="w-full h-screen text-center">
<div className="w-full h-full text-center">
Copy link
Contributor

Choose a reason for hiding this comment

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

혹시 무슨 차이가 있는지 아실까요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

아 h-screen은 뷰포트 높이를 말하는 것이고 h-full은 부모 엘리먼트의 높이를 따라가게 한 차이입니다!

Copy link
Collaborator

Choose a reason for hiding this comment

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

이게 전역에 있는 때는 안되다가 이 위치에 넣어서 되는건가요?
아니면 제가 텍스트를 일단 띄워놔서 안됐던 건가요??

Copy link
Contributor Author

Choose a reason for hiding this comment

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

네 맞습니다!! 전역에 있는 loading.tsx가 채팅방에 접속할때는 적용이 안되는 것 같더라고요! 그래서 임시방편으로 일단 채팅방 페이지에도 loading.tsx를 넣어준 것입니다. 이 원인이 채팅방 SSR과 관련이 있는 부분일까요?

Copy link
Collaborator

Choose a reason for hiding this comment

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

모르겠네요 알아봐야 할 것 같습니다

@juyeon-park juyeon-park merged commit c93af22 into develop Nov 29, 2023
3 of 4 checks passed
@juyeon-park juyeon-park deleted the NABI-359 branch November 29, 2023 08:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants