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

✨ 업로드 및 validation 로직 리팩토링 등 #163

Merged
merged 5 commits into from
Jan 4, 2024

Conversation

oaoong
Copy link
Collaborator

@oaoong oaoong commented Jan 2, 2024

- 목적

관련 티켓 번호: 385, 387, 392


- 주요 변경 사항

  • 385: 제 로컬에서 뭐가 꼬인 것인지 다크모드가 또 말썽이어서 제대로 비활성화 했습니다. 배포 버전이 이상 없는 것으로 봐서는 또 파일명 이슈인지 잘 모르겠네요
  • 387: 업로드 컴포넌트를 tanstack query를 사용하여 리팩토링하고, 업로드 중일 때 안내 메세지를 추가했습니다.
  • 392: 기존 validation에서 과도한 useEffect 내부 로직을 간소화하고, 의존성에 따라 분리하여 작성했습니다. 불필요한 state를 삭제하고 useCallback을 이용한 렌더링 최적화를 진행했습니다.

기타 사항 (선택)

  • 원래 fork repo에서 PR을 날리기로 했는데, 제 레포가 배포 중인 레포이기도 하고, 파일명 이슈가 왠지 있는 것 같아 main-jaehee <- develop-jaehee로 PR을 생성하게 됐습니다.
  • 때문에 develop-jaehee에서 작업하고 각 티켓은 커밋 단위로 작업했습니다. 양해 부탁드립니다. (다른 의견이나 문제 있으면 언제든지 알려주세요)

- 스크린샷 (선택)

@oaoong oaoong added the D-2 label Jan 2, 2024
@oaoong oaoong self-assigned this Jan 2, 2024
@oaoong oaoong merged commit c517c08 into main-jaehee Jan 4, 2024
1 check passed
@oaoong oaoong deleted the develop-jaehee branch January 4, 2024 04:52
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.

1 participant