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

Init: 폴더구조 세팅 #16

Merged
merged 4 commits into from
Jan 7, 2025
Merged

Init: 폴더구조 세팅 #16

merged 4 commits into from
Jan 7, 2025

Conversation

gwagjiug
Copy link
Member

@gwagjiug gwagjiug commented Jan 6, 2025

📌 Summary

📚 Tasks

  • 폴더 구조 수정하였습니다.

👀 To Reviewer

기존 폴더구조를 수정하였습니다. 기획 피그마 상에 유저 플로우를 고려해서 큰 단위로 우선 pages 폴더를 만들어서 구분지었고, 나중에 추가적으로 설명이 더 필요할 것 같아요

도메인 단위로 큰 폴더가 존재하고, 안에 세부적인 page가 존재합니다 그리고 해당 도메인 하위에 해당하는 모든 page의 hook & constants 들을 도메인 폴더에 생성하면 됩니다.

shared 폴더는 기존 보편적인 폴더구조에 common 에 해당하는 폴더라고 생각하시면 됩니다.

참고한 아티클이나 폴더 구조 수정 근거에 관한 것들은 한번 회의하면서 공유하면 좋을 것 같아요

폴더명, 파일명에 케밥 케이스를 적용한 것은 민하 & 지욱 의견입니다 뭔가 좀 더 직관적인 것 같고 꼭 카멜케이스를 써야하나? 의문을 가진 것도 있어요 의견 부탁드립니다.

📸 Screenshot

케밥 케이스vs 카멜 케이스

image

image

@gwagjiug gwagjiug added ⚙️ Setting 개발 환경 세팅 지욱 🥁 지욱 labels Jan 6, 2025
@gwagjiug gwagjiug self-assigned this Jan 6, 2025
@gwagjiug gwagjiug requested a review from a team as a code owner January 6, 2025 15:27
@gwagjiug gwagjiug requested review from seueooo, m2na7, daahyunk and bongtta and removed request for a team January 6, 2025 15:27
@gwagjiug gwagjiug linked an issue Jan 6, 2025 that may be closed by this pull request
Copy link
Member

@m2na7 m2na7 left a comment

Choose a reason for hiding this comment

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

지욱님만의 철학과 충분한 근거가 담긴 구조같아서 좋아요 ~ 👍

Copy link
Member

@daahyunk daahyunk left a comment

Choose a reason for hiding this comment

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

케밥 케이스 쓰는 거 좋은 방법인 것 같아요 제 눈에도 더 명시적이네용 👍 수고하셨습니다~

Copy link
Contributor

@seueooo seueooo left a comment

Choose a reason for hiding this comment

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

저는 익숙치 않은 방식인데 가독성 면에서 확실히 나은 것 같네요 LGTM !!!

@gwagjiug gwagjiug merged commit 930162d into develop Jan 7, 2025
@gwagjiug gwagjiug deleted the init/folder/#5 branch January 7, 2025 08:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚙️ Setting 개발 환경 세팅 지욱 🥁 지욱
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Init]: 폴더구조 세팅
4 participants