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

Migrate /common-space to react-hook-form #1202

Merged
merged 12 commits into from
Nov 26, 2024

Conversation

ChaeyeonAhn
Copy link
Contributor

@ChaeyeonAhn ChaeyeonAhn commented Nov 12, 2024

요약 *

It closes #1199

공용공간 비정기사용 신청 폼을 리액트 훅 폼으로 마이그레이션!

  • Dev mode test (Third Frame에서 Post 요청 잘 되는지 확인 필요)
  • local mock mode test

마이그레이션 자체 외의 주요 변경 사항

- SecondFrame에서 모달을 닫을 시, (onConfirm) dateTimeRange를 undefined로 초기화하는 코드 삽입

스크린샷

2024-11-19.2.46.24.mov

이후 Task *

  • 없음

@ChaeyeonAhn ChaeyeonAhn linked an issue Nov 12, 2024 that may be closed by this pull request
2 tasks
@ChaeyeonAhn ChaeyeonAhn changed the title feat: start migration Migrate /common-space to react-hook-form Nov 12, 2024
@ChaeyeonAhn ChaeyeonAhn self-assigned this Nov 12, 2024
@ChaeyeonAhn ChaeyeonAhn added enhancement New feature or request front-end labels Nov 12, 2024
@ChaeyeonAhn ChaeyeonAhn marked this pull request as ready for review November 18, 2024 18:03
@wjeongchoi
Copy link
Contributor

마지막 페이지에서 동아리 이름이 안 들어가는 것 같아요

Copy link
Contributor

@wjeongchoi wjeongchoi left a comment

Choose a reason for hiding this comment

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

api 문제로 secondFrame 내용을 못 봐서 수정 되면 마저 볼게요!

@ChaeyeonAhn
Copy link
Contributor Author

마지막 페이지에서 동아리 이름이 안 들어가는 것 같아요

요것두 해결했습니다! 감사합니다ㅎㅎㅎ

@ChaeyeonAhn
Copy link
Contributor Author

11/20 19:36 기준

phoneInput disabled 문제는 현재 third frame에서 마지막에 예약 내역 조회를 할 때 유저의 전화번호를 폼에서 가져오는 게 아니라 디비에서 가져옵니다 그래서 disabled로 막아놓게 하는 것 같아요 (그냥 디비에서 불러온 걸로 쓰도록)

그리고 third frame에 동아리 이름 반영 안 되는 거 + 마지막에 예약 내역 조회할때 지금까지 선택한 거 다 반영되게 하는거 완료했습니다!

API만 확인하면 될 것 같아요ㅎㅎ 감사합니닷!
@wjeongchoi

Copy link
Contributor

@babycroc babycroc left a comment

Choose a reason for hiding this comment

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

대충 코멘트에 적은 거 읽어보면 좋을듯! 글구 리뷰 rerequest 해줭~

@ChaeyeonAhn
Copy link
Contributor Author

11/20 수 22:56

일단 getCommonSpaces 파싱 제거 -> 2번째 프레임에서 에러 나는 건 없어진 것 같아요
그런데 마지막에 제출할 때 백에서 student.student_id를 집어넣어야 하는데 �student_id가 undefined여서 문제가 생기는 것 같아요

@ChaeyeonAhn
Copy link
Contributor Author

ChaeyeonAhn commented Nov 20, 2024

자자 마지막 업데이트!!

결과적으로 마지막 제출 안 되는 건 백 문제였어요 (데브 디비에 경우에 해당하는 row가 없어서 불러오는 게 없었음)

그래서 다른 걸로 불러봤더니 됩니다
어쨌든 디비 문제인데
그래서 테스트 잘 굴러가게 하려고 백 코드 하나를 바꿔놨어요 (common-space.controller.ts)
확인하시면 다시 돌려놓겠습니다! :)

Outdated. 밑의 comment 봐 주세요!

감사합니다아아

@ChaeyeonAhn
Copy link
Contributor Author

허허허 진짜진짜 마지막 Update

common-space-controller 코드를 GetStudent에서 가져온 user.studentId를 쓰도록 했더니 굉장히 잘 됩니다
우와아

Copy link
Contributor

@babycroc babycroc left a comment

Choose a reason for hiding this comment

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

오! 오! 오! 잘 된당
백엔드 고친거 그냥 같이 머지하면 될 것 같구

후속 작업으로 몇 개만 더 부탁을 하자면.......

  • 마지막 페이지에서 "다음" 버튼 "신청" 버튼으로 바꾸기
  • 마지막 페이지 "신청" 버튼 누르면 요 모달 띄우고 "확인" 버튼 누르면 "마이페이지"로 redirect 시키기
  • console에서 나는 에러 고치기
image

그러구 나서 쭉 QA 돌아보면 되겠다! 짱이야 짱

@ChaeyeonAhn
Copy link
Contributor Author

2024-11-26.2.25.38.mov

오예 다 고쳤어여!

@babycroc

바로 머지해도 되려나.. 무서우니까 하루 기다리기

@ChaeyeonAhn ChaeyeonAhn merged commit 548146e into dev Nov 26, 2024
1 check passed
@ChaeyeonAhn ChaeyeonAhn deleted the 1199-use-react-hook-form-for-common-space branch November 26, 2024 16:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request front-end
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use react-hook-form for /common-space
3 participants