Feature/auth routing#21
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
| className="w-full" | ||
| onError={(err) => { | ||
| if (isApiError(err)) { | ||
| setError(err.message) |
There was a problem hiding this comment.
우선 백엔드에서 던진 메세지를 그대로 전달하는군요
백엔드에서 어떤 메세지를 던지는지도 확인해봐야겠네요
There was a problem hiding this comment.
MVP구현단계에선 빠른 출시,개발이 중요해서 콘솔로그나 서비스가 터지지않는 에러처리를 하고 있지만, 여유가 생긴다면 logger를 도입해서 로깅시스템과 대시보드 등을 구축하면 좋을 것 같습니다.
백엔드는 당연히 해야하는게 맞고, 프론트도 하면 좋을 것 같아요.
| </p> | ||
| <h1 | ||
| className="mt-4 font-heading font-extrabold text-sage-900 leading-[1.05]" | ||
| style={{ fontSize: 'clamp(36px, 5vw, 52px)', letterSpacing: '-0.5px' }} |
There was a problem hiding this comment.
이 부분은 tailwind 대신 인라인 스타일을 쓰셨군요
임시라면 상관없겠지만 하나로 통일하는게 좋아보입니다.
There was a problem hiding this comment.
이 부분은 애매하긴 하네요. 사실 taliwind의 한계때문에 이렇게 써서 tailwind로 사용하려면 토큰화 시켜야하긴하는데 이부분은 좀 더 생각해보겠습니다.
| const { user, logout } = useAuth() | ||
| const [loggingOut, setLoggingOut] = useState(false) | ||
|
|
||
| const handleLogout = async () => { |
There was a problem hiding this comment.
handleLogout을 여러 곳에서 사용하니 커스텀 훅으로 빼면 깔끔하겠네요
There was a problem hiding this comment.
클린코드 좋네요. 커스텀 훅으로 분리했습니다.
한 가지 더 고려한 부분은 useAuth에 합치지 않고 전역 상태와 Mutation 상태의 책임을 분리하는 방향으로 리팩토링했습니다.
2 participants
변경 사항
📌 작업 목적
구현된 인증 로직을 전역 라우터와 앱 최상단에 연결하여 전체 인증 흐름을 완성합니다.
🛠️ 주요 변경 사항
Login,Workspace,AuthCallback페이지 껍데기(Shell) 추가app/router/index.tsx에 라우트 트리 정의 및RequireAuth가드 적용app/providers에AuthProvider를 주입하고main.tsx진입점에 연결widgets/site-nav사이드 네비게이션에 로그인 상태 연동현재 파악된 문제