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

♻️ Refactor: 페이지 컴포넌트 파일명, 컴포넌트명 통일 (XXXPage -> Page) #23

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

yeonhee7935
Copy link
Collaborator

@yeonhee7935 yeonhee7935 commented Jan 18, 2025

작업 내용

  • 폴더, 파일, 컴포넌트명 통일

이러한 변경이 이루어지는 이유는 무엇인가요?

  • 폴더명 변경
    • store -> shop: zustand를 관리하는 store폴더와 중복되어 헷갈릴 수 있다는 피드백이 있었음.
    • 매장정보 관리 페이지를 store에서 shop으로 변경
    • 라우팅도 이에 맞게 변경
  • 폴더명, 페이지 컴포넌트명 통일
    • XXXPage -> Page
    • 페이지 컴포넌트는 default로 export
    • nextjs의 라우팅 파일과 비슷한 방식으로 변경
  • 예시
// /src/pages/login/page.tsx
export default function Page(){
  ....
}

테스트 방법

변경사항에 대해 리뷰어가 테스트할 수 있는 가이드를 step by step으로 제공하세요.

  1. 내용

병합 전 체크리스트

  • 수정 내용에 오타나 컨벤션이 틀린 부분이 없는지 확인했나요?
  • 추가된 리뷰에 대해 모두 수정 및 재리뷰를 완료했나요?

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