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

[노철] - 비지니스 로직 분리 #471

Merged
merged 8 commits into from
Feb 5, 2024
Merged

Conversation

qkdl60
Copy link
Contributor

@qkdl60 qkdl60 commented Feb 4, 2024

📌 이슈 번호

🚀 구현 내용

  • 마이페이지, 계획, 계획 수정 페이지 비지니스 로직 분리
  • 로그인 페이지에 불필요한 훅 제거

@qkdl60 qkdl60 added the 🚨긴급 긴급한 확인 필요 label Feb 4, 2024
@qkdl60 qkdl60 added this to the V1.1 milestone Feb 4, 2024
@qkdl60 qkdl60 requested review from MinwooP and suehdn February 4, 2024 16:22
@qkdl60 qkdl60 self-assigned this Feb 4, 2024
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.

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

Copy link
Contributor

Choose a reason for hiding this comment

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

함수별로 한줄씩 띄워주는 것도 가독성에 좋을 것 같습니다 !!

Comment on lines +16 to +41
const {
receiveType,
nickname,
remindEmail,
defaultEmail,
emailVerified,
isChangeReceiveTypePending,
isOpenEmailModal,
isOpenLogOutModal,
isOpenRemindWayModal,
isOpenWithdrawalModal,
remindWay,
handleChangeNickName,
handleChangeReceiveType,
handleCloseEmailVerificationModal,
handleCloseLogOutModal,
handleCloseWithdrawalModal,
handleGORemindWay,
handleGoEmailVerification,
handleLogOut,
handleRealLogOut,
handleRealWithdrawal,
handleSetVerifiedEmail,
handleWithdrawal,
setIsOpenRemindWayModal,
} = useMyPage();
Copy link
Contributor

Choose a reason for hiding this comment

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

ㅋㅋㅋㅋㅋ 어마무시하군요

@qkdl60 qkdl60 merged commit fbd16d2 into dev Feb 5, 2024
@qkdl60 qkdl60 deleted the feat/#468/page-refactoring branch February 5, 2024 05:12
suehdn added a commit that referenced this pull request Feb 5, 2024
* ♻️ : #460 - 페이지 파일, 컴포넌트 파일 내 import문 index 파일 import 하도록 경로 수정

* ♻️ : #460 - hooks 폴더 내 index 파일 정의

* ♻️ : #460 - hooks/apis 폴더 내 index 파일 정의

* ♻️ : #460 - utils 폴더 내 index 파일 정의 및 decideRamdomIconNumber 유틸함수 삭제

* ♻️ : #467 - 주소 확인 정규식 const로 분리

* ♻️ : #467 - navigation 비즈니스 로직 분리

* ♻️ : #467 - 사용하지 않는 header 컴포넌트 삭제

* ♻️ : #467 - tabprops 타입 분리

* ♻️ : #467 - tab 컴포넌트 비즈니스 로직 분리

* ♻️ : #467 - ExplorePlans 비즈니스 로직 분리

* ♻️ : #467 - NewPlan 컴포넌트 비즈니스 로직 분리

* ♻️ : #467 - ProgressBarProps 타입 분리

* ♻️ : #467 - ProgressBar 컴포넌트 비즈니스 로직 분리

* ♻️ : #467 - default function으로 변경

* ♻️ : #467 - MyPlan 컴포넌트 비즈니스 로직 분리

* ♻️ : #467 - home 비즈니스 로직 분리

* ♻️ : #467 - feedback/[planId] 비즈니스 로직 분리

* ♻️ : #467 - EvaluateRadio 컴포넌트 비즈니스 로직 분리

* ♻️ : #467 - feedback/evaluate 비즈니스 로직 분리

* ♻️ : #467 - 빌드오류 해결

* ✨ : #472 - PR시 build 작업 추가 (CI)

* [노철] - 비지니스 로직 분리  (#471)

* ♻️ : #468 - 마이페이지 로직 분리

* ♻️ : #468 - 계획 페이지 로직 분리

* ♻️ : #468 - 계획 수정 페이지 로직 분리

* ♻️ : #468 - 필요없는 훅 제거

* ♻️ : #468 - 함수 대신 컴포넌트 return 변경

* ♻️ : #468 - 함수 대신 컴포넌트로 변경

* ♻️ : #474 - app 파일 구조 변경

* 🐛 : #472 - 브랜치명 develop -> dev로 변경

* 🐛 : #472 - node 버전 20으로 변경

* ✨ : #477 - 포크 자동화 ACTION 추가

---------

Co-authored-by: 박민우 <[email protected]>
Co-authored-by: 박민우 <[email protected]>
Co-authored-by: 박노철 <[email protected]>
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