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/#205] 회의 확정시 토큰 삭제 및 입력 단계에서 큐카드로 리다이렉트 #206

Merged
merged 3 commits into from
Nov 3, 2023

Conversation

simeunseo
Copy link
Member

#205

todo

  • 회의 확정시 토큰 삭제
  • 확정된 회의의 가능시간 입력 페이지 (:auth/schedule/:meetingId) 접근시 alert 띄운 뒤 큐카드로 리다이렉트
  • 확정된 회의의 우선순위 입력 페이지 (:auth/priority/:meetingId) 접근시 alert 띄운 뒤 큐카드로 리다이렉트

📌 내가 알게 된 부분

  • try-catch에서 error가 났을 때 alert를 띄우니 alert가 두번씩 뜨는 문제가 발생했습니다. 렌더링이 두번 되는건가 싶어서 useEffect랑 useMemo랑 막 건드려봤는데... (지금 스스로 렌더링 방식에 대한 이해가 너무 부족한게 느껴지더라고요...) 그냥 React.StrictMode때문에 그런것이었습니다..!!
  • React.StrictMode는 언세이프한 코드 감지하기 위해, 개발모드에서 의도적으로 마운팅을 두 번 일으킵니다. 프로덕션 빌드에는 영향을 끼치지 않습니다.

📌스크린샷

2023-10-16.9.31.44.mov

@simeunseo simeunseo added feat new feature 은서 은서의 개발 라벨 labels Oct 16, 2023
@simeunseo simeunseo self-assigned this Oct 16, 2023
@vercel
Copy link

vercel bot commented Oct 16, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
asap ✅ Ready (Inspect) Visit Preview Oct 16, 2023 0:32am

@ljh0608
Copy link
Member

ljh0608 commented Oct 16, 2023

배워갑니다 LGTM !!

@ljh0608 ljh0608 merged commit 1b860c5 into develop Nov 3, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat new feature size/M 은서 은서의 개발 라벨
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants