-
Notifications
You must be signed in to change notification settings - Fork 2
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
[노철] - 비지니스 로직 분리 #471
Conversation
…into feat/#468/page-refactoring
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.
깔끔하게 잘 분리하신 것 같습니다! 고생하셨습니다 👏🏻
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.
함수별로 한줄씩 띄워주는 것도 가독성에 좋을 것 같습니다 !!
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(); |
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.
ㅋㅋㅋㅋㅋ 어마무시하군요
…into feat/#468/page-refactoring
* ♻️ : #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]>
📌 이슈 번호
🚀 구현 내용