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.

Sign up for GitHub

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

Jump to bottom

[혜수] - 비즈니스 로직 분리 #470

Merged
merged 16 commits into from
Feb 5, 2024
Merged

[혜수] - 비즈니스 로직 분리 #470

merged 16 commits into from
Feb 5, 2024

Conversation

suehdn
Copy link
Contributor

@suehdn suehdn commented Feb 4, 2024

📌 이슈 번호

close #467

🚀 구현 내용

  • 담당 페이지 비즈니스 로직 분리 작업 완료

📘 참고 사항

  • export default function ~~ 방식으로 작성했습니다.
    image
    각 hooks 파일에 index.ts를 두었습니다.

@suehdn
♻️ : #467 - 주소 확인 정규식 const로 분리
99108db
@suehdn
♻️ : #467 - navigation 비즈니스 로직 분리
51c8c91
@suehdn
♻️ : #467 - 사용하지 않는 header 컴포넌트 삭제
57bfcf1
@suehdn
♻️ : #467 - tabprops 타입 분리
9417a36
@suehdn
♻️ : #467 - tab 컴포넌트 비즈니스 로직 분리
51cb3e9
@suehdn
♻️ : #467 - ExplorePlans 비즈니스 로직 분리
5fc4a07
@suehdn
♻️ : #467 - NewPlan 컴포넌트 비즈니스 로직 분리
55efafe
@suehdn
♻️ : #467 - ProgressBarProps 타입 분리
67427ed
@suehdn
♻️ : #467 - ProgressBar 컴포넌트 비즈니스 로직 분리
5a72f11
@suehdn
♻️ : #467 - default function으로 변경
711d524
@suehdn
♻️ : #467 - MyPlan 컴포넌트 비즈니스 로직 분리
c6f77c1
@suehdn
♻️ : #467 - home 비즈니스 로직 분리
26c6fd8
@suehdn
♻️ : #467 - feedback/[planId] 비즈니스 로직 분리
b624973
@suehdn
♻️ : #467 - EvaluateRadio 컴포넌트 비즈니스 로직 분리
193f5dc
@suehdn
♻️ : #467 - feedback/evaluate 비즈니스 로직 분리
eb83f16
@suehdn suehdn added the 💡중간 보통단계 label Feb 4, 2024
@suehdn suehdn requested review from MinwooP and qkdl60 February 4, 2024 15:41
@suehdn suehdn self-assigned this Feb 4, 2024
qkdl60
qkdl60 approved these changes Feb 4, 2024
View reviewed changes
Copy link
Contributor

@qkdl60 qkdl60 left a comment

Choose a reason for hiding this comment

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

혜수님 고생하셨습니다. 저는 index 없이 그냥 export default 해서 가져왔는데, 혜수님 처럼 훅이 많다면 혜수님 방식이 더 좋아보이네요 👍

MinwooP
MinwooP approved these changes Feb 5, 2024
View reviewed changes
Copy link
Contributor

@MinwooP MinwooP left a comment

Choose a reason for hiding this comment

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

깔끔하게 잘 분리하신 것 같습니다! 고생하셨습니다 👏🏻

@suehdn suehdn merged commit 86de908 into dev Feb 5, 2024
@suehdn suehdn deleted the feat/#467/business-logic branch February 5, 2024 05:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@qkdl60 qkdl60 qkdl60 approved these changes

@MinwooP MinwooP MinwooP approved these changes

Assignees

@suehdn suehdn

Labels
💡중간 보통단계
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[혜수] - 비즈니스 로직 분리
3 participants
@suehdn @MinwooP @qkdl60