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

[김민찬] Sprint9 #296

Conversation

itscold96
Copy link
Collaborator

@itscold96 itscold96 commented Aug 16, 2024

주요 변경사항

  • 지난 Sprint9 미션 내용에서 피드백 받은 내용을 반영하여 리팩토링을 진행하였습니다.
  1. Tanstack-Query의 query-client를 Layout 컴포넌트로 옮겨서 전역적으로 사용할 수 있게 하였습니다.
  2. Boards 컴포넌트 내부를 모듈화하여 가독성과 유지보수성을 개선하였습니다.
  3. Next/Image가 생기면서 리액트에서 만들어 두었던 (Custom)Image 컴포넌트가 필요없어졌기에 삭제하였습니다.
  4. getArticles 요청 함수의 파라미터의 page, size 타입을 number로 고치고,
    URLSearchParams API를 사용하여 URL 쿼리스트링의 유지보수성을 개선하였습니다.
  5. getArticles 요청 함수에 try catch 문을 추가하여 네트워크 에러에도 페이지가 정상적으로 작동하도록 개선하였습니다.
  6. styled-component, 반응형 쿼리 사이즈를 상수로 하여 JS로 처리하였던 것을 mixin으로 변경하여 SCSS로 처리하도록 수정하였습니다.
  7. reset.css를 추가하였습니다.
  8. 매번 window 객체를 사용할 때, 동적로딩을 불러오는 것이 오히려 보일러 플레이트를 만들 수도 있겠다는 생각에 동적 로딩 로직을 없애고,
    window 객체의 유무에 따라 useEffect를 분기처리하도록 하였습니다.
  9. useWindow 훅에서 어차피 너비 외에는 사용하지 않기에, useInnerWidth 훅으로 변경하고 window.innerWidth만을
    반환하도록 개선하였습니다.
  10. 페이지 어디서든 사용할 수 있도록 정렬 드롭다운을 모듈화하였습니다.

멘토에게

  • 다음주는 개인 프로젝트 설계 후, 프로젝트 세팅, 메인 페이지 작업까지 해서 개인 레포지토리 링크를 남겨보도록 하겠습니다. 감사합니다:)

Copy link
Collaborator

@lisarnjs lisarnjs left a comment

Choose a reason for hiding this comment

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

깔끔한 리팩토링이였습니다~~!
역시 민찬님 최고에요 👍 👍
민찬님 저희 회사에 데려가고 시파요 아주 ㅎㅁㅎ
이번 한주도 화이팅이에요!

@@ -17,19 +17,19 @@
background-color: var(--white);
border-bottom: 1px solid var(--border-gray);

@media #{$tabletSize} {
@include tablet {
Copy link
Collaborator

Choose a reason for hiding this comment

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

오호 좋아요~~! 이제 scss 간지나게 사용하시겠군요 :)

@lisarnjs lisarnjs merged commit 8c378d6 into codeit-bootcamp-frontend:Next-김민찬 Aug 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants