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: 폴더 구조 변경 및 react-query 구조 개선 #56

Merged
merged 34 commits into from
Jul 25, 2023

Conversation

ojj1123
Copy link
Collaborator

@ojj1123 ojj1123 commented Jun 18, 2023

📮 관련 이슈

🌱 PR 포인트

📸 스크린샷

  • 폴더 구조 변경(src 폴더 내 폴더 구조 변경)
AS-IS TO-BE
스크린샷 2023-07-22 22 12 27 스크린샷 2023-07-22 22 12 51

ojj1123 added 8 commits June 13, 2023 01:50
- components/common -> common/components
- 공통 컴포넌트 이동하면서 추가적으로 생긴 경로 변경 반영
- application/hooks/common -> common/hooks
- 공통 hook 이동하면서 추가적으로 생긴 경로 변경 반영
- application/util -> common/utils
- 경로 수정으로 추가적인 변경 반영
- infra/sdk -> common/libs
- 경로 변경으로 인한 추가적인 경로 수정
- twin.macro 를 import하는 파일 목록을 src 폴더 내에 있는 모든 폴더로 수정
- https://github.com/ben-rogerson/twin.examples/blob/master/next-emotion/withTwin.js
@ojj1123 ojj1123 added the refactor Refectoring label Jun 18, 2023
@ojj1123 ojj1123 self-assigned this Jun 18, 2023
@vercel
Copy link

vercel bot commented Jun 18, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
thismeme-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 22, 2023 2:59pm

ojj1123 and others added 12 commits June 24, 2023 16:35
- /components/home -> /features/home/components
- deadcode 삭제
- /components/meme -> /features/memes/components
- deadcode 삭제
- feature간에 공통으로 쓰이는 컴포넌트는 /features/common 에 배치
- /components/mypage -> /features/mypage/components
- /components/collect -> /features/collect/components
- /components/search -> /features/search/components
- /components/tags -> /features/explore/tags/components
- /components/keywords -> /features/explore/keywords/components
- TagCategory 컴포넌트는 공통 컴포넌트로 /feature/common 으로 이동
- /components/share -> /features/share/components
- /components/hocs -> /features/common/hocs
- /application/hooks/domain -> /features/common/hooks
- /application/hooks/domain -> /features/common/hooks
@ojj1123 ojj1123 changed the title Refactor/issue 53 react-query 구조 개선 및 비즈니스 로직 관심사 모으기 Jul 17, 2023
ojj1123 added 5 commits July 18, 2023 14:11
- /application/hooks/api/core -> /api/core
- useCoreInfiniteQuery 추후 리팩터링 시 삭제 예정
- /infra/api/index.ts -> /api/core/axios.ts
- /infra/api/index.ts 삭제 예정
- /application/hooks/api/meme -> /api/meme
- /application/hooks/api/tags -> /api/tag
- /application/hooks/api/collection -> /api/collection
ojj1123 added 4 commits July 22, 2023 00:11
- /application/hooks/api/search -> /api/search
- useGetUserFindMemes(회원이 찾는 밈 API) 삭제(dead code)
- /application/hooks/api/account -> /api/account
- /application -> /api/core
- DefaultPageProps 타입을 /api/core/query-client.tsx 로 이동
- /api/core/axios.ts 로 이동함
@ojj1123 ojj1123 marked this pull request as ready for review July 22, 2023 13:13
@ojj1123 ojj1123 requested review from elbica and SeieunYoo as code owners July 22, 2023 13:13
- 관리 되지 않는 Navigation, Category, Sidebar 스토리북 삭제
- 위 스토리북 코드에서 오류 발생으로 삭제함
- preview.js에 OverlayProvider 추가
@ojj1123
Copy link
Collaborator Author

ojj1123 commented Jul 22, 2023

스토리북 빌드 오류 발생

github action

스크린샷 2023-07-23 00 02 51

스토리북 빌드 결과
스크린샷 2023-07-22 23 59 31

스토리북 빌드 오류 해결 방식

  • Masonry 스토리북 해결 -> 내부에서 useOverlay 사용하고 있어서 OverlayProvider 추가해줌
  • Navigation, Category, Sidebar 스토리북
    • 관리도 안되고 있고, 오류가 발생해서 삭제했습니다.
    • 디자인 시스템 만들 때 추가하면 될 듯합니다!

@ojj1123 ojj1123 changed the title react-query 구조 개선 및 비즈니스 로직 관심사 모으기 ♻️Refactor: 폴더 구조 변경 및react-query 구조 개선 Jul 22, 2023
@ojj1123 ojj1123 changed the title ♻️Refactor: 폴더 구조 변경 및react-query 구조 개선 ♻️Refactor: 폴더 구조 변경 및 react-query 구조 개선 Jul 22, 2023
@ojj1123 ojj1123 changed the title ♻️Refactor: 폴더 구조 변경 및 react-query 구조 개선 ♻️ Refactor: 폴더 구조 변경 및 react-query 구조 개선 Jul 22, 2023
Copy link
Member

@SeieunYoo SeieunYoo left a comment

Choose a reason for hiding this comment

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

수고하셨습니다🤗 infra 계층 api 는 따로 관리하는 걸까요?!

@ojj1123
Copy link
Collaborator Author

ojj1123 commented Jul 25, 2023

수고하셨습니다🤗 infra 계층 api 는 따로 관리하는 걸까요?!

@SeieunYoo 따로 관리하는건 아니고 인증/인가 설계랑 엮기는 부분이 있어서 일단 그대로 두었습니다~

기능상 문제는 없어서 일단은 두었고 다시 리팩터링을 해보려고 합니다

@ojj1123 ojj1123 merged commit 344211a into develop Jul 25, 2023
@ojj1123 ojj1123 deleted the refactor/issue-53 branch July 25, 2023 10:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor Refectoring
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants