Skip to content

Feature/430 remove antd#431

Merged
choiseona merged 15 commits intodevfrom
feature/430-remove-antd
Dec 24, 2024
Merged

Feature/430 remove antd#431
choiseona merged 15 commits intodevfrom
feature/430-remove-antd

Conversation

@choiseona
Copy link
Member

🚨 Pull request 전에 확인해야 할 사항( [x] 체크해주세요 )

  • 반드시 확인할 것!
    pull a topic/feature/bugfix branch 에서 dev branch로 PR요청하는지 확인하세요.
    master/main에 하시면 안됩니다.

  • 커밋 또는 모든 커밋의 메시지 스타일이 convention과 일치하는지 확인해주세요.

  • PR작업에 대한 오른쪽의 라벨을 붙여주세요.

📌 개요

  • 비효율적인 라이브러리 직접 구현 후 삭제
  • 안쓰는 라이브러리 삭제
  • 이미지 포맷, 사이즈 제한

closes #430

✨ 작업 내용

  • 기존의 antd를 사용해서 만든 것을 직접 구현하고 antd를 종속성에서 제거했습니다.
    • 저희 프로젝트에서 antd를 전체적으로 사용하지 않고 부분만 사용하고 있는 상황입니다.
    • 이 상황에서 크기가 큰 라이브러리인 antd를 그대로 유지하는 것이 비효율적이라고 생각하여 제거했습니다.
  • react hook form을 도입함에 따라 기존에 사용했던 formik, yup을 삭제했습니다.
    • react hook form 만으로는 코드가 지저분해 yup을 같이 사용하는 것을 고려중입니다.
  • antd를 없앰에 따라 datePicker 라이브러리를 따로 설치하였습니다.
    • datePicker로 가장 많이 쓰이는 라이브러리를 설치하였습니다. 추가적으로 해당 라이브러리르 사용하면서 date-fns도 필요한데 이것이 moment를 대체할 수 있어 moment도 추가적으로 삭제해주었습니다.
    • moment보다 date-fns가 훨씬 가볍습니다.
  • 쓰이지 않는 react-modal, modal 라이브러리를 제거했습니다.
  • 백엔드 api 로 보내게되는 이미지 포맷을 jpg, jpeg, png, gif로 만 가능하도록하였고, 사이즈는 5MB로 제한하였습니다.

📸 스크린샷(선택)

🔊 리뷰어에게(선택)

📚 추가 사항(선택)

@choiseona choiseona added the enhancement New feature or request label Dec 24, 2024
@choiseona choiseona requested a review from songmok December 24, 2024 16:53
@choiseona choiseona self-assigned this Dec 24, 2024
@vercel
Copy link

vercel bot commented Dec 24, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
gitget ⬜️ Ignored (Inspect) Visit Preview Dec 24, 2024 5:00pm

@choiseona choiseona merged commit a0de841 into dev Dec 24, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant