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

[PEAUTY-157] /designer/signup-detail 페이지 컴포넌트 분리 및 API 연결 #72

Merged
merged 17 commits into from
Dec 10, 2024

Conversation

ZUITOPIA
Copy link
Contributor

@ZUITOPIA ZUITOPIA commented Dec 7, 2024

💻 작업 내용

작업 내용 1

제목, 내용 input이 연달아서 필요할 때 사용할 TitleContentInput 컴포넌트 생성 후 분리
image
사용 예시
image

작업 내용 2

ShopInfoInputSection 컴포넌트 생성 후 분리
image

작업 내용 3

CustomInput 컴포넌트 수정 (extraText를 props로 전달하여 아래와 같이 사용 가능)
(전)
image
변경 전 코드
image

(후)
image
사용 예시 (변경 후 코드)
image

작업 내용 4

자격증 입력받는 부분 CertificateInputSection 컴포넌트로 분리
image

작업 내용 5

결제 방식 중복선택 가능하므로 radio button에서 multi select button으로 변경
(전)
image
(후)
image

작업 내용 6

react-daum-postcode 사용하여 우편번호 검색 서비스 띄우기
image

이 과정에서 GNB.styles.ts의 Nav의 z-index를 1000에서 ${theme.zIndex.gnb}; 로,
CustomModal.styles.ts의 ModalBackdrop의 z-index를 1000 에서 ${theme.zIndex.bottomSheet}; 로 수정하였습니다

🧠 PR 체크

완료하셨다면 띄어쓰기 대신 [] 사이에 소문자 x로 표시해주세요.

  • 담당자와 리뷰어를 설정했어요.
  • label을 설정했어요.

@ZUITOPIA ZUITOPIA added the 모두가 봐야해 모두의 리뷰를 받자 label Dec 7, 2024
@ZUITOPIA ZUITOPIA self-assigned this Dec 7, 2024
Copy link
Contributor

@myoungjinGo-FE myoungjinGo-FE left a comment

Choose a reason for hiding this comment

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

PR에 대한 자세한설명 좋아요 👍 이해완

@ZUITOPIA ZUITOPIA changed the title [PEAUTY-157] TitleContentInput 컴포넌트 생성 [PEAUTY-157] /designer/signup-detail 페이지 컴포넌트 분리 및 API 연결 Dec 8, 2024
@ZUITOPIA ZUITOPIA changed the title [PEAUTY-157] /designer/signup-detail 페이지 컴포넌트 분리 및 API 연결 [PEAUTY-157] /designer/signup-detail 페이지 컴포넌트 분리 Dec 9, 2024
@ZUITOPIA ZUITOPIA changed the title [PEAUTY-157] /designer/signup-detail 페이지 컴포넌트 분리 [PEAUTY-157] /designer/signup-detail 페이지 컴포넌트 분리 및 API 연결 Dec 9, 2024
Copy link
Contributor

@Sieonn Sieonn left a comment

Choose a reason for hiding this comment

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

충돌 해결하고 push 한 다음 merge해주세요

@myoungjinGo-FE myoungjinGo-FE changed the base branch from sprint/1 to develop December 10, 2024 04:33
@ZUITOPIA ZUITOPIA merged commit d37498f into develop Dec 10, 2024
2 checks passed
@ZUITOPIA ZUITOPIA deleted the feature/PEAUTY-157 branch December 10, 2024 08:25
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.

3 participants