Skip to content

Auth 컴포넌트 작성, 필요 라이브러리 설치 #55

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
Merged
hanana1253 merged 14 commits into from
Mar 23, 2022
Merged

Auth 컴포넌트 작성, 필요 라이브러리 설치 #55

hanana1253 merged 14 commits into from
Mar 23, 2022

Conversation

@hanana1253
Copy link
Member

@hanana1253 hanana1253 commented Mar 22, 2022 •
edited
Loading

PR Type

What kind of change does this PR introduce?

  • 버그를 수정했어요.
  • 새로운 기능을 추가했어요.
  • 코드 스타일 업데이트를 했어요(포맷팅, 지역변수)
  • 리팩토링을 했어요 (기능적인 변화 없이, api 변경 없이)
  • 환경설정을 변경했어요.
  • 문서 내용을 변경했어요.
  • 기타 사항을 설명해 주세요.

Related Issues

#36

What does this PR do?

  • Auth 컴포넌트 작성에 필요한 라이브러리 Formik, yup을 설치했습니다.
  • currentForm에 따라 변경되는 내용의 Auth 컴포넌트 작성
  • type과 style, stories 파일 작성
  • firebase 로직을 한스푼 프로젝트에서 거의 그대로 옮기고 타입 정의만 몇개 추가했습니다.

Other information

  • 이제 auth관련 함수 및 Dialog 컴포넌트를 작성해야 하겠습니다.

@hanana1253
refactor: Pagination 현재 페이지 계산을 useMemo로 메모했습니다.
eba76cf 
- currentPage, totalResults, limit 등이 바뀔 때만 새로 계산되도록 적용
- 허슬데브  정민님의 코멘트 반영
@hanana1253
feat: Auth 컴포넌트 추가 및 이를 위한 패키지(Formik, yup) 설치
e6f3294 
DONE:
- currentForm에 따라 필드 및 submit 함수가 달리 작동하는  Auth 컴포넌트를 만들었습니다.

TODO:
- field라는 값이 string으로 인식되어 타입에러가 나는데 이 부분을 해결해야합니다.
- 스타일링을 해야 합니다.
@hanana1253
conf: Formik, yup 설치
cda3787 
설치로 인해 yarn.lock 파일이 바뀌었습니다.
@hanana1253
Merge branch 'feature/auth' into develop
72cc149
@hanana1253
fix: Styled 파일과 types 파일 잘못 기재한 부분 수정했습니다.
9ef6f47 
- 파일명을 제대로 안 보고 작성해버렸습니다 죄송합니다.
@hanana1253 hanana1253 added 🌟 feature 새로운 기능을 만들어요! ⚙️ conf 환경 설정 세팅이에요 ✈️ migration 마이그레이션 작업을 진행해요! 🧱 component 컴포넌트 단위 작업을 진행해요! labels Mar 22, 2022
@hanana1253 hanana1253 added this to the Sprint2 milestone Mar 22, 2022
@hanana1253 hanana1253 self-assigned this Mar 22, 2022
@hanana1253
conf: gitignore 파일 수정 (firebase관련 파일 추가)
5940faf 
- firebase 관련 설정파일을 gitignore에 추가
@hanana1253
feat: AuthService에 SignIn, SignUp 함수 연결
6fe939e 
- firebase의  signin/signup 함수를 기존 한스푼 프로젝트에서 가져왔습니다.
hhhyyo
hhhyyo reviewed Mar 23, 2022
View reviewed changes
hustle-dev
hustle-dev reviewed Mar 23, 2022
View reviewed changes
hhhyyo
hhhyyo requested changes Mar 23, 2022
View reviewed changes
jhyj0521
jhyj0521 reviewed Mar 23, 2022
View reviewed changes
@hanana1253
Merge branch 'develop' of https://github.com/TeamCooks/TwoSpoon into …
d727495 
…develop
@hanana1253
chore: Prop type에 스타일을 위한 $warning 프롭 타입 추가
0d33141 
- 타입 오류 안나도록 프롭 타입 추가했습니다.
@hanana1253
chore: AuthService 파일에 as const로 객체 형태를 고정시켰습니다.
ece8ecf 
- signin/signup 에 필요한 메시지 상수들이라서 객체 형태를 타입 고정했습니다.
@hanana1253
fix: utils import 경로 수정
270a377 
- import구문에서 절대경로로 alias가 되어있는 것을 적용
@hanana1253
refactor: usePageNum 훅 작성
6f8d37f 
- pageNum을 계산하는 usePageNum 훅을 작성했습니다.
@hanana1253
fix: stories 파일들에서 decorator 제거
8bc11c4 
- 효원님이 preview에 decorator 일괄 적용해주셔서 각 stories 파일에서는 삭제했어요.
@hanana1253
refactor: 필요없는 변수 제거
1ea30cc 
- errorCode라는 변수를 굳이 안 만들고 바로 error.code로 에러 throw
@hanana1253 hanana1253 merged commit 350753b into TeamCooks:develop Mar 23, 2022
This was referenced Mar 25, 2022
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hustle-dev hustle-dev hustle-dev left review comments

@jhyj0521 jhyj0521 jhyj0521 left review comments

@hhhyyo hhhyyo hhhyyo requested changes

Assignees

@hanana1253 hanana1253

Labels

🧱 component 컴포넌트 단위 작업을 진행해요! ⚙️ conf 환경 설정 세팅이에요 🌟 feature 새로운 기능을 만들어요! ✈️ migration 마이그레이션 작업을 진행해요!

Projects

None yet

Milestone

Sprint2

Development

Successfully merging this pull request may close these issues.

4 participants

@hanana1253 @hhhyyo @hustle-dev @jhyj0521