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

[비즈니스] step 상태에 따라 버튼 색상 변경 #345

Merged
merged 3 commits into from
Jun 5, 2024

Conversation

chaeseungyun
Copy link
Contributor

  • Close #ISSUE_NUMBER

What is this PR? 🔍

  • 기능 : step 상태에 따라 버튼 색상 변경
  • issue : #

Changes 📝

isStepComplete 혹은 isComplete 일때 버튼이 active 됩니다.

ScreenShot 📷

Test CheckList ✅

  • test 1
  • test 2
  • test 3

Precaution

✔️ Please check if the PR fulfills these requirements

  • It's submitted to the correct branch, not the develop branch unconditionally?
  • If on a hotfix branch, ensure it targets main?
  • There are no warning message when you run yarn lint

@chaeseungyun chaeseungyun self-assigned this Jun 5, 2024
@github-actions github-actions bot requested review from hoooooony and KimKyungYun June 5, 2024 07:17
[styles.button__active]: !!errors.root,
[styles.button__active]: isStepComplete || isComplete,
Copy link
Contributor

Choose a reason for hiding this comment

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

__active 보다 --active는 어떠신가요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

수정했습니다

@chaeseungyun chaeseungyun merged commit 02e8c60 into feat/SMS Jun 5, 2024
@chaeseungyun chaeseungyun deleted the feat/fix_common_component branch June 5, 2024 07:23
hoooooony added a commit that referenced this pull request Jun 20, 2024
* [비즈니스] 회원가입, 비밀번호 찾기 공통 레이아웃 완성 (#341)

* feat: auth 단계를 조절하는 훅 추가

* feat: 비밀번호 찾기, 회원가입 공통 레이아웃 추가

* feat: 회원가입, 비밀번호 찾기 완성 페이지 추가

* feat: 공통 레이아웃 적용

* refactor: outlet에 props로 steps 전달

* fix: 오타 수정

* feat : 이전 회원가입 폴더명 변경, 동의 페이지 추가

* feat : 스테틱한 내용 파일 경로 수정

* feat : useStep에서 현재 단계 상태를 추가하여 다음 페이지 로직 통제 (#344)

Co-authored-by: (hoooooony) <([email protected])>

* 버튼 전역적으로 선언된 스타일 변경

* feat : 1페이지 ui 완성

* [비즈니스] step 상태에 따라 버튼 색상 변경 (#345)

* feat:  step 상태에 따라 button 색상 변경

* refactor: 이전 페이지로 돌아갈 때 isStepComplete 복구

* refactor: className 변경

* feat : 뒤로가기 대응

* fix : 동의상태관리 오류 수정

* [비즈니스] 비밀번호 찾기 문자인증 도입 (#349)

* feat: 비밀번호 찾기 초기 페이지 구성

* feat: 비밀번호 찾기 문자인증 api 생성

* feat: 문자 인증 response 인터페이스 추가

* feat: 비밀번호 찾기 문자 인증 페이지 완성

* refactor: 비밀번호 변경 api 생성 및 타입 생성

* feat: 비밀번호 변경 페이지 완성

* feat: 비밀번호 찾기 페이지 추가

* refactor: warning 아이콘 추가

* chore: scss 파일 위치 수정

* chore: 불필요한 파일 삭제 및 적용

* refactor: import cycle 해결

* fix: lint 에러 수정

* refactor: 사용자 입력으로 오류 수정 시 에러 초기화

* feat: useDebounce 훅 추가

* refactor: 오류 처리 및 useDebouce 적용

* refactor: 코드 가독성 향상

* feat : 스텝 2 추가

* feat : 컴포넌트 분리 및 스텝2 api 연결

* feat: step2 완성

* feat : 모달 추가

* feat  : modal 오류 수정

* refactor: 문자 인증 후 accessToken 저장

* feat: 다중 이미지 업로드 구현

* feat :step3 완성

* fix : 사업자 번호 오류 수정

* fix : 가게 선택 로직 오류 수정

* feat : 가게선택 css 추가

* feat: 전화번호 로그인 구현 및 적용

* fix: 경로 오타 수정

* fix: 디바운싱 추가 및 파라미터 변경

* feat : 오너 정보 등록 다음 단계 로직

* fix: 컨플릭트 해결

---------

Co-authored-by: chaeseungyun <[email protected]>
Co-authored-by: (hoooooony) <([email protected])>
Co-authored-by: chaeseungyun <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants