-
Notifications
You must be signed in to change notification settings - Fork 0
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: 공통 컴포넌트, hook 이동 #54
Conversation
- components/common -> common/components - 공통 컴포넌트 이동하면서 추가적으로 생긴 경로 변경 반영
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
- application/hooks/common -> common/hooks - 공통 hook 이동하면서 추가적으로 생긴 경로 변경 반영
- application/util -> common/utils - 경로 수정으로 추가적인 변경 반영
- infra/sdk -> common/libs - 경로 변경으로 인한 추가적인 경로 수정
빌드 에러 발생하는거 해결하고 있으니 다른거 먼저 리뷰 해주세요~ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
짱짱짱
import { Button } from "../../Button"; | ||
import { Icon } from "../../Icon"; | ||
import { RandomImage } from "../../RandomImge"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
여기서는 절대경로 적용 안되나유 ? @common/components/~
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
vscode 자동완성 import 로 저게 나와서 그렇긴 합니다
common/components 폴더 안에 있는 컴포넌트 끼리는 절대경로가 추천이 안되더라구요 ㅜㅜ
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
도메인 컴포넌트들은 component/도메인
폴더 밑에 있는 것 같네요!
추후 테스트나 도메인 hook, 유틸도 고려한다면
feature/도메인/
밑에 component 폴더를 만드는게 관심사가 더 가깝지 않을까요??
- feature/memes/component
- feature/memes/hooks
- feature/memes/tests
- feature/memes/container (페이지 컴포넌트를 feature와 가깝게 두기 위해 container를 만들기도 함)
아직 feature 폴더 작업은 안했어요~ |
- twin.macro 를 import하는 파일 목록을 src 폴더 내에 있는 모든 폴더로 수정 - https://github.com/ben-rogerson/twin.examples/blob/master/next-emotion/withTwin.js
📮 관련 이슈
⛳️ 작업 내용
🌱 PR 포인트
refactor/issue-53
마더 브랜치를 만든 후 이 브랜치에 RFC: FDA(Feature-Driven Architecture) 기반 폴더 구조 변경 및 react-query 구조 개선 #53 에 대한 작업을 모두 완료 한 후 최종적으로 develop으로refactor/issue-53
브랜치를 merge 합니다.components/common
->common/components
application/hooks/common
->common/hooks
application/util
->common/utils
infra/sdk
->common/libs