-
Notifications
You must be signed in to change notification settings - Fork 2
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
[Refactor] lottie-react 재도입 및 patch를 이용한 번들 사이즈 감소 #466
Conversation
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍👍👍👍
Deploying sopt-recruiting-frontend-test with Cloudflare Pages
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
오 patch package 처음 알게되었는데 해당 라브 설치에 쓰이는 비용보다 이를 통해 라브 최적화한 효과가 더 큰 경우 너무 요긴한 방법이네요!! 덕분에 또 배워갑니다
이거 한번 설치했으니 앞으로 다른 라이브러리 번들 사이즈도 이 친구 통해서 최적화 시키기 좋겠네요
고생하셨습니다! 👍 👍
Related Issue : Closes #459
🧑🎤 Summary
🧑🎤 Screenshot
[수정 전 전체 번들사이즈]
[수정 후 전체 번들사이즈]
[수정 전 lottie 번들사이즈]
[수정 후 lottie 번들사이즈]
🧑🎤 Comment
lottie-web/light 버전을 이용하기 위해 lottie-react에서 lottie-web으로 migration을 해줬는데요
그럴 필요 없이 patch를 이용해서 library를 수정할 수 있더라고요
yarn v2 이상에선 yarn patch를 이용해주면 되었지만
yarn v1를 사용하고 있었기에 patch-package를 이용해줬어요
node_modules/lottie-react 에 있는 파일을 수정해준 뒤
yarn patch-package lottie-react
를 통해 patch 해주었어요똑같이 300KB 정도 세이브 해줄 수 있었습니다 :)