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

refactor : 리스트 아이템 컴포넌트 수정&리팩토링 #462

Merged

Conversation

cmlim0070
Copy link
Collaborator

🚀요약

리스트 아이템 컴포넌트 리팩토링

📸사진 (구현 캡처)

📝작업 내용

  • 보관함에서 사용되는 각 편지 종류별 타입을 인터페이스로 재정의했습니다
  • 카테고리 렌더링 부분을 따로 분리했습니다
  • 패턴매칭을 굳이 사용할 필요가 없는듯 하여 ts-pattern으로 작성되어있던 네비게이트 url 생성 함수를 switch-case로 변경했습니다
  • 보관함 url 파라미터명을 lettertype에서 selectedLetterType으로 변경했습니다
  • 기타 파일들 - 타입명 변경에 따른 타입 삭제, 수정

🔍백엔드 전달 사항

🎸기타 (연관 이슈)

StorageList 컴포넌트도 현재 책임이 너무 많아서 추후에
체크박스 매니저
삭제 모달 관리
두가지 로직을 분리하려고 합니다

close #451

@cmlim0070 cmlim0070 added the ♻️ refactor 리펙토링 label Jan 2, 2025
@cmlim0070 cmlim0070 self-assigned this Jan 2, 2025
@cmlim0070 cmlim0070 linked an issue Jan 2, 2025 that may be closed by this pull request
1 task
Copy link

github-actions bot commented Jan 2, 2025

✅ Build 성공!

Copy link

github-actions bot commented Jan 2, 2025

⚡️ Lighthouse Report (Average of 5 runs)

Category Score
🟠 Performance 80
🟠 Accessibility 68
🟢 SEO 90
🟢 Best Practices 100

Core Web Vitals (Average)

Metric Value
🔴 First Contentful Paint 3.6 s
🟠 Largest Contentful Paint 3.8 s
🟢 Cumulative Layout Shift 0.0

Copy link
Member

@HelloWook HelloWook left a comment

Choose a reason for hiding this comment

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

확인했습니다! 고생하셨습니다!

@cmlim0070 cmlim0070 merged commit b7b3bb9 into main Jan 3, 2025
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.

refactor : 리스트 아이템 컴포넌트 리팩토링
2 participants