[Refactor/#340] 공연 수정하기 코드 리팩토링 - Githubissues

TEAM-BEAT / BEAT-Client

BE AT WHERE YOUR HEART BEATS, BEAT

https://www.beatlive.kr

9 stars 4 forks source link

[Refactor/#340] 공연 수정하기 코드 리팩토링 #353

Closed ocahs9 closed 2 weeks ago

ocahs9 commented 3 weeks ago

📌 관련 이슈번호

Closes #340

🎟️ PR 유형

어떤 변경 사항이 있나요?

[ ] 새 기능 추가
[x] 버그 수정
[ ] CSS 등 사용자 UI 디자인 변경
[x] 리팩토링

✅ Key Changes

이번 PR에서 작업한 내용을 간략히 설명해주세요

작업 내용

📢 To Reviewers

useReducer로 복잡한 상태 관리를 시도했습니다.
dataState, modifyState 라는 이름으로 2개의 state를 분리하여, api 요청 시 필요한 state , 그 외 수정하기 페이지에서 필요한 state 로 분리했습니다.
IsBookerExist를 직접 이용하는 로직, data로부터 받아온 state를 직접 관리하고 api 요청 보내는 로직(form으로 관리하는 것이 아니라)으로 수정했습니다.
공연 삭제하기 버튼을 누른 즉시, 삭제 가능 여부를 판단하도록 로직을 변경했습니다.
자잘한 버그들 수정했습니다 (commit 내역 참고)
데이트피커, 타임 피커는 공통 컴포넌트로 사용하는 것이 더 좋을 것 같아 직접 구현은 하지 않았습니다.

📸 스크린샷

🔗 참고 자료

github-actions[bot] commented 3 weeks ago

PR 작성하느라 고생 많았어요!! 라벨 잘 지정되었는지 확인 한 번 해 주기 🫶

sonarcloud[bot] commented 2 weeks ago

Quality Gate failed

Failed conditions
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarCloud

Catch issues before they fail your Quality Gate with our IDE extension SonarLint