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/6 ] 폴더구조 및 테스트 파일 분리 작업 #9

Merged
merged 15 commits into from
Jun 3, 2024
Merged

Conversation

Happhee
Copy link
Collaborator

@Happhee Happhee commented Jun 3, 2024

🔥 Related Issues

resolve #6

💜 작업 내용

  • src로 폴더 경로 수정
  • alias 경로 설정
  • 기본 테스트 파일 생성 및 관리방법 모색

✅ PR Point

prettier 설정 쌍따옴표로 변경하였습니다.

"singleQuote": false,

create-component.js

pnpm run create {컴포넌트명}

작성하면 shadcn/ui 컴포넌트 쉽게 깔려용!! 명령어 하도 길어서 따로 만들었습니다!

api 폴더

  • api-config.ts 👉 axios instance 관리하는파일인데, fetch로 통신하는게 좋을지...는! 더 찾아봐야할것같습니다!
  • query-client.ts 👉queryclient 하나로 관리하면 좋을것같아서 만들었습니다.

src 하위경로로 폴더 모두 이동했습니다! 😆

test파일 관리

  • 공통 👉 component이든 utils이든, page 테스트이든 __test__폴더 하위로 관리하도록 설정했습니다.
    @soomin9106 요렇게 묶는게 맞는지 확인부탁드려요! 🙇‍♀️

📚 Reference

@Happhee Happhee requested a review from soomin9106 June 3, 2024 01:10
@soomin9106
Copy link
Collaborator

soomin9106 commented Jun 3, 2024

storybook 이랑 패키지 컨플릭 나는 듯 합니당!!
이것만 해결해서 머지 하면 좋을 것 같아요

Copy link
Collaborator

Choose a reason for hiding this comment

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

확인했습니당 ㅎㅎ

Copy link
Collaborator

Choose a reason for hiding this comment

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

오 이거 아주 좋네요!! 굿

@Happhee
Copy link
Collaborator Author

Happhee commented Jun 3, 2024

@soomin9106 넵!! conflic 해결하고 머지 할게용!

Copy link

cloudflare-workers-and-pages bot commented Jun 3, 2024

Deploying 24th-web-team-1-fe with  Cloudflare Pages  Cloudflare Pages

Latest commit: d042f3c
Status: ✅  Deploy successful!
Preview URL: https://ae2d898c.24th-web-team-1-fe.pages.dev
Branch Preview URL: https://init-6.24th-web-team-1-fe.pages.dev

View logs

@Happhee Happhee merged commit 238b82c into main Jun 3, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[ 초기셋팅 ] 폴더구조 및 테스트 파일 분리 작업
2 participants