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/#274 SSR에서 토큰 탑재 자동화 및 마이페이지 리팩토링 #279

Merged
merged 7 commits into from
Jan 25, 2024

Conversation

navyjeongs
Copy link
Member

@navyjeongs navyjeongs commented Jan 18, 2024

🤠 개요

  • 마이페이지 리팩토링
  • 쿼리 상수 파일
  • SSR에서 토큰 탑재 자동화 함수 생성

💫 설명

  • 마이페이지에 로직을 커스텀훅으로 분리하였어요.

  • 마이페이지에서 낙관적 업데이트를 적용해서 이메일 전송이 되기전 전송되었다고 알려줘요

  • 또, 쿼리와 뮤테이션을 관리하는 상수 파일을 만들었어요. 앞으로 작성하는 쿼리와 뮤테이션 키와 함수는 해당 파일에 작성해서 불러와주세요!

  • useInput 커스텀훅을 만들었어요.

  • SSR에서 헤더에 토큰이 자동적으로 탑재되도록 HOF를 하나 만들었어요.
    기존에는 getServerSideProps를 사용하는 페이지마다 토큰을 탑재해야했는데 해당 불편함을 해결했어요.

📷 스크린샷 (Optional)

@navyjeongs navyjeongs added the ✨ Feat 기능 개발 label Jan 18, 2024
@navyjeongs navyjeongs requested a review from pp449 January 18, 2024 04:18
@navyjeongs navyjeongs self-assigned this Jan 18, 2024
@navyjeongs navyjeongs linked an issue Jan 18, 2024 that may be closed by this pull request
2 tasks
Copy link
Member

@pp449 pp449 left a comment

Choose a reason for hiding this comment

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

✅ LGTM
일정이 있어서 이제 확인했네요!

@navyjeongs navyjeongs merged commit 7629f0c into dev Jan 25, 2024
1 check passed
@navyjeongs navyjeongs deleted the Feat/#274 branch January 25, 2024 08:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feat 기능 개발
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feat: 마이페이지의 SSR에서 react-query 사용
2 participants