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

[회원가입] 회원가입 페이지 API 연동 #34

Merged
merged 47 commits into from
Jan 6, 2024
Merged

[회원가입] 회원가입 페이지 API 연동 #34

merged 47 commits into from
Jan 6, 2024

Conversation

HAEROOL
Copy link
Contributor

@HAEROOL HAEROOL commented Dec 13, 2023

[#2 ] request

회원가입 페이지의 회원가입 기능을 완료했습니다.
가게 검색의 경우 모바일 뷰가 구현되지 않아 구현 완료되면 추가로 적용하겠습니다.
가게 검색에서 선택된 가게는 가게 이름과 전화번호 외에도 shop_id를 추가적으로 저장하도록 변경했습니다.

  • 사장님 데이터(가게 이름, 대표자 이름, 가게 등록 번호, 대표자 전화번호, 관련 파일 업로드 ) 회원가입 데이터에 추가
  • 입력한 데이터 기반으로 회원가입 기능

Please check if the PR fulfills these requirements

  • It's submitted to develop branch, not the main branch
  • The commit message follows our guidelines
  • Did you merge recent deveop branch?

Screenshot

스크린샷 2023-12-14 오전 1 50 54 스크린샷 2023-12-14 오전 1 52 14 스크린샷 2023-12-14 오전 1 51 26

Precautions (main files for this PR ...)

HAEROOL added 30 commits November 1, 2023 18:30
@HAEROOL
Merge pull request #15 from BCSDLab/feature/add-signup-page
35d9228 
[회원가입] 회원가입 페이지 유저 데이터 입력 로직 구현 및 api 연동
@HAEROOL
Merge branch 'feature/add-signup-page' of https://github.com/BCSDLab/…
6fa6184 
…KOIN_OWNER_WEB into feature/add-signup-page
@HAEROOL
feat: 개인 연락처 input 추가
cdc2244
@HAEROOL
feat: 가게 이름 입력 input 추가
ef51715
@HAEROOL
style: 가게 검색 버튼 모바일, 데스크톱 스타일 수정
82125fe
@HAEROOL
feat: 가게 이름 검색 UI 추가
0a5b040
@HAEROOL
feat: 가게 검색 Toast UI 추가
bf5fcf6
@HAEROOL
style: Toast 스타일 추가
e4d390d
@HAEROOL
style: 상점 찾기 모달 스타일 수정
dda2b83
@HAEROOL
feat: 가게 검색 기능 중 클릭 시 검색 기능 추가
68b32f3
@HAEROOL
style: 토스트 팝업 open 시 상점 스크롤 height 축소
3e5fd54
@HAEROOL
feat: 상점 검색 검색 인풋 엔터 이벤트 추가
85cfc68
@HAEROOL
feat: 사장님 정보 입력 인풋 에러 핸들링 추가
0f9c995
@HAEROOL
feat: 회원가입 사장님 정보 input 에러 메시지 구현
9f1248d
@HAEROOL
Merge branch 'develop' into feature/add-signup-page
f8667e0
@HAEROOL
Merge remote-tracking branch 'origin/feature/#27' into feature/add-si…
e03d101 
…gnup-page
@HAEROOL
refactor: store 이름 shop으로 통일
88b477e
@HAEROOL
feat: 사업자 등록증 파일 업로드 로직 추가
db5c11f
@HAEROOL
feat: 사업자 등록증 파일 삭제 로직 추가
878ce3a
@HAEROOL
style: 사업자 등록증 업로드 div height, width 수정
369930a
@HAEROOL
feat: 사업자 등록증 드래그앤드롭 파일 업로드 구현
99e5ccd
@HAEROOL
Merge branch 'develop' into feature/add-signup-page
411414a
@HAEROOL
style: Owner 데이터 입력 step height 삭제
d0d44cb
@HAEROOL
feat: 업로드 파일 관리를 위한 useFileController 훅 추가
d5a6d3d
@HAEROOL
feat: 상점검색 모달 회원가입 페이지와 연결
6442b90
@HAEROOL
refactor: 사업자 등록증 input, 대표자 번호 input register 결합
8c69e13
@HAEROOL
feat: 사업자 등록번호와 전화번호 autofocus 구현
63cf2b1
@HAEROOL
feat: mobile 뷰 등록번호, 연락처 input register 추가
c563bb2
@HAEROOL
feat: 사장님 정보 입력 다음 버튼 활성화 로직 추가
ea31d69
@HAEROOL
fix: input 입력없이 next 버튼 눌러지는 버그 수정
c3acc22
@HAEROOL HAEROOL added the 🌏 Deploy 배포 관련 label Dec 13, 2023
MinGu-Jeong
MinGu-Jeong reviewed Dec 18, 2023
View reviewed changes
Copy link
Contributor

@MinGu-Jeong MinGu-Jeong left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!
store 변수명으로 쓰인 곳이 너무 많아 shop변수명으로 통일하는 것이 쉽지 않을것 같단 생각이 듭니다...🥲
관련하여 추후 회의때 다시 한 번 얘기해보는 것도 괜찮을 것 같기도 합니다.

daepan
daepan requested changes Dec 19, 2023
View reviewed changes
Copy link
Contributor

@daepan daepan left a comment

Choose a reason for hiding this comment

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

수고하셨습니다. 코멘트 한번 확인해주시고 수정해주시면 될 것 같네요

dooohun
dooohun reviewed Dec 19, 2023
View reviewed changes
Copy link
Contributor

@dooohun dooohun left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!

hanagertrudeKim
hanagertrudeKim requested changes Jan 2, 2024
View reviewed changes
Copy link
Contributor

@hanagertrudeKim hanagertrudeKim left a comment

Choose a reason for hiding this comment

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

수고많으셧습니다~ 리뷰가 늦었네요
코드량이 전체적으로 많아서 리뷰하는데 조금 시간이 걸렸습니다ㅎㅎ
구조적으로 리팩토링할게 있어보이지만, 우선 빠르게 리뷰들 확인해주세요!

hanagertrudeKim
hanagertrudeKim approved these changes Jan 4, 2024
View reviewed changes
Copy link
Contributor

@hanagertrudeKim hanagertrudeKim left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!

@hoooooony
Copy link
Contributor

수고하셨습니다!!

@HAEROOL HAEROOL requested a review from daepan January 6, 2024 10:55
@HAEROOL HAEROOL merged commit 56bb3ca into develop Jan 6, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MinGu-Jeong MinGu-Jeong MinGu-Jeong left review comments

@dooohun dooohun dooohun left review comments

@hanagertrudeKim hanagertrudeKim hanagertrudeKim approved these changes

@hoooooony hoooooony Awaiting requested review from hoooooony

@daepan daepan Awaiting requested review from daepan

Assignees

@HAEROOL HAEROOL

Labels
🌏 Deploy 배포 관련
Projects
No open projects
KOIN_OWNER WEB
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@HAEROOL @hoooooony @daepan @MinGu-Jeong @dooohun @hanagertrudeKim