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/#1] 프로젝트 초기 환경 설정 #2

Merged
merged 11 commits into from
Jan 13, 2025
Merged

[Feat/#1] 프로젝트 초기 환경 설정 #2

merged 11 commits into from
Jan 13, 2025

Conversation

Jxxunnn
Copy link
Member

@Jxxunnn Jxxunnn commented Jan 12, 2025

이슈 번호

closes #1

작업한 내용

  • Next.js 14 설치

    • 공식문서 기준으로 14.2.23 버전 명시하여 설치
  • React Query v5 설치

    • QueryClientProvider 적용
    • ReactQueryDevtools 적용
  • styled-components v6 설치

    • 서버 렌더링 관련 설정 적용 공식문서 참고
    • 브라우저 스타일 초기화 위해 styled-reset 적용
  • Axios 설치

    • fetch 기반 어댑터로 Axios 인스턴스 추가
    • API 기본 URL을 .env.example 파일에 환경 변수로 추가
  • ESLint, Prettier 설치

    • Next.js 기본 제공 ESLint 사용
    • Prettier와 충돌 방지를 위해 eslint-plugin-prettier 적용

작업 결과

.

비고

MSW 설치 ❌
MSW가 기본적으로 app router랑 호환되지 않는 것 같더라구. 관련 이슈를 확인했어. 개발 종속성이기도 하고 rotue handlers로 대체할 수 있을 것 같아서 이번 작업에서 MSW를 적용할 수 있는 방법을 추가로 찾지는 않았어.

폴더 구조
작업을 진행하다 보니 폴더를 생성할 필요가 생겨서 라우트 기반과 피쳐 기반 폴더 구조 사이에서 고민하다가 우리 서비스는 페이지가 많을 것 같지 않아서 피쳐 기반 폴더 구조로 틀을 잡아두었어. 관련해서 다이어그램을 작성해봤는데 아직 디테일은 정리되지 않은 상태야. 괜찮으면 이 구조를 기반으로 발전시키고 더 좋은 아이디어가 있다면 같이 논의해보면 좋을 것 같아!

@Jxxunnn Jxxunnn added the enhancement New feature or request label Jan 12, 2025
@Jxxunnn Jxxunnn self-assigned this Jan 12, 2025
@Jxxunnn Jxxunnn requested review from a team and ljh0608 and removed request for a team January 12, 2025 13:11
@Jxxunnn Jxxunnn removed the enhancement New feature or request label Jan 12, 2025
@Jxxunnn
Copy link
Member Author

Jxxunnn commented Jan 13, 2025

@ljh0608 같이 검토한 PR이라 병합 할게. #4 배포 설정 작업에 필요해서!

@Jxxunnn Jxxunnn merged commit 965d322 into main Jan 13, 2025
@Jxxunnn Jxxunnn deleted the feat/#1 branch January 13, 2025 12:27
Copy link

@ljh0608 ljh0608 left a comment

Choose a reason for hiding this comment

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

LGTM

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.

[기능] 프로젝트 환경 설정
2 participants