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: 라우팅 개선 #91

Merged
merged 3 commits into from
Dec 29, 2024
Merged

Conversation

lgrin-byte
Copy link
Member

💡 다음 이슈를 해결했어요.

  • app/components/RequireAuth/RequireAuth.tsx 에서는 최대한 로그인 유무만 판단할 수 있도록 하였고
  • 생성, 삭제와 같은 api 응답시 라우팅 처리부분을 개선하여 깜빡임 최소화함
  • 힌트폰 배경화면 배포시 나가기 위해 타겟 브랜치는 힌트폰 배경화면 작업브랜치로 설정함



✅ 셀프 체크리스트

  • 브랜치 전략에 맞는 브랜치에 PR을 올리고 있습니다. (master/main이 아닙니다.)
  • 커밋 메세지를 컨벤션에 맞추었습니다.
  • 변경 후 코드는 컴파일러/브라우저 warning/error 가 발생시키지 않습니다.
  • 변경 후 코드는 기존의 테스트를 통과합니다.
  • 테스트 추가가 필요한지 검토해보았고, 필요한 경우 테스트를 추가했습니다.
  • docs 수정이 필요한지 검토해보았고, 필요한 경우 docs를 수정했습니다.

@lgrin-byte lgrin-byte changed the title Feat/refactor router feat: 라우팅 개선 Dec 25, 2024
@lgrin-byte lgrin-byte merged commit 9322c60 into feat/theme-image-preview Dec 29, 2024
2 checks passed
@lgrin-byte lgrin-byte deleted the feat/refactor-router branch December 29, 2024 06:26
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.

1 participant