Closed p-acid closed 1 year ago
작업 범위에 대해 간략하게 작성합니다.
리뷰어가 중점적으로 봐야 하는 부분을 바로 알 수 있도록 변경된 내용을 나열합니다.
<SiteHeader />
/api/auth/[...nextauth]/route.ts
redirect
/waiting
getSessionAccount
findFirst
session
null
refresh
/waiting/page.tsx
useForm
useQuery
defaultValue
undefined
useEffect
PR에 대한 추가 설명이나 작업하면서 고민이 되었던 부분 등
/(auth)/feed/page.tsx
'use client'
🛠️ 작업 내용 (Content)
📝 상세 설명
<SiteHeader />
영역 우측에 임시 로그아웃 버튼을 추가했습니다./api/auth/[...nextauth]/route.ts
파일의redirect
설정으로 로그인 및 로그아웃 하는 경우 해당 경로로 이동되어/waiting
에서 분기를 처리하도록 했습니다.getSessionAccount
함수를 통해 불러온 로그인 유저의 정보를 통해 분기하여 리다이렉트 시킵니다.getSessionAccount
부분에서findFirst
로 무작위 첫 번째 레코드를 받아오던 문제를session
이 없는 경우null
을 반환하는 것으로 처리했습니다.refresh
를 통해 서버 컴포넌트(/waiting/page.tsx
) 업데이트 실행useForm
사용을 위해 클라이언트 컴포넌트로 활용되기에useQuery
를 통한 클라이언트 요청 및 캐시 업데이트 적용useForm
의defaultValue
가 페칭할 때 기준으로undefined
이기에 데이터가 성공적으로 요청되면useEffect
실행⚙️ 기타 사항
/(auth)/feed/page.tsx
파일이'use client'
로 클라이언트 컴포넌트 처리가 되어 있어 몬스터 리스트 요청 전에 로그아웃을 실행하면 에러가 발생합니다. 해당 부분'use client'
가 사라지면 해결될 문제라 일단 보류하였습니다.🚨 Merge 전 필요 작업 (Checklist before merge)