Skip to content

Conversation

@ParkSohyunee
Copy link
Contributor

개요

  • 코드 품질 관리를 위해 eslint 경고가 발생하는 일부 컴포넌트의 코드를 정리했습니다.

작업 사항

  • 사용하지 않는 변수, import문 제거 (@typescript-eslint/no-unused-vars 경고)
  • any 타입을 구체적인 타입으로 수정 (@typescript-eslint/no-explicit-any 경고)
  • useEffect 의존성 배열에 변수 추가 (react-hooks/exhaustive-deps 경고)
  • (구) 콜렉션 페이지 제거
  • 사용하지 않는 파일 제거
  • 불필요한 주석 정리

참고 사항 (optional)

  • home 페이지, search 페이지, notices 페이지 및 관련 컴포넌트는 추후 작업 진행 예정입니다.

리뷰어에게

  • 시간 나실 때 확인 부탁드립니다. 💕

@vercel
Copy link

vercel bot commented Dec 17, 2024

@ParkSohyunee is attempting to deploy a commit to the Eujin Ahn's projects Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Contributor

@seoyoung-min seoyoung-min left a comment

Choose a reason for hiding this comment

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

소현님 깔끔한 코드 정리로 경고까지 처리해 주셔서 너무 감사합니다! 확인이 늦었네요ㅠㅠ LGTM!!

"framer-motion": "^11.0.6",
"html-to-image": "^1.11.11",
"http-proxy-middleware": "^2.0.6",
"lottie-react": "^2.4.0",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

📌 기존 로딩 UI를 위해 설치했던 react-lottie 라이브러리가 더이상 관리되지 않는 점과 빌드 시 지속적으로 발생하는 버전이슈 문제로 인해 lottie-react 라이브러리로 대체하였습니다. 보여지는 이미지는 동일하고 라이브러리만 교체된 점 참고 부탁드립니다.

@ParkSohyunee ParkSohyunee merged commit 9c1d1e7 into 8-Sprinters:dev Jan 2, 2025
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Refactor 리팩토링

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants