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

전체적인 Router 수정 #112

Merged
merged 11 commits into from
Oct 25, 2024
Merged

전체적인 Router 수정 #112

merged 11 commits into from
Oct 25, 2024

Conversation

Dobbymin
Copy link
Contributor

#️⃣ 연관된 이슈

ex) #이슈번호, - #이슈번호

📝 작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요.(이미지 첨부 가능)

  • 시니또, 보호자로 구분하여 페이지 경로 수정
  • 상수화한 페이지 path 수정
  • 전체적인 css 수정
  • 폴더 구조 수정

스크린샷 (선택)

💬 리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

⏰ 현재 버그

✏ Git Close

close #이슈번호

@Dobbymin Dobbymin added the ♻️ Refactoring 코드 리팩토링 & 클린 코드 작업을 진행하는 경우 label Oct 25, 2024
@Dobbymin Dobbymin requested review from Diwoni and JYN523 October 25, 2024 09:15
@Dobbymin Dobbymin self-assigned this Oct 25, 2024
@Dobbymin Dobbymin linked an issue Oct 25, 2024 that may be closed by this pull request
Copy link
Collaborator

@Diwoni Diwoni left a comment

Choose a reason for hiding this comment

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

전체적으로 index.ts 파일이나 경로가 깔끔하게 정리되었네요.. 눈 아프셨을 텐데 고생 많으셨습니다!

라우터 부분은 봤을 때 제 부분은 문제 없는 것 같습니다!

],
},
{
// TODO: 콜백에 들어가야 할지 가이드라인에 들어가야 할지 잘 모르겠음.
Copy link
Collaborator

Choose a reason for hiding this comment

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

흠 제 생각에는 콜백에 들어가는 게 흐름 상 괜찮을 것 같습니다.

queryClient,
AuthProvider,
globalStyle,
} from '@/shared';
Copy link
Collaborator

Choose a reason for hiding this comment

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

이 부분이 수정 안됐었군요..! 훨씬 깔끔한 것 같습니다!

@Dobbymin Dobbymin merged commit 4e2d020 into Weekly Oct 25, 2024
1 check passed
@Dobbymin Dobbymin deleted the Fix/issue-#109 branch October 25, 2024 13:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
♻️ Refactoring 코드 리팩토링 & 클린 코드 작업을 진행하는 경우
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Router 경로 수정
2 participants