Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
관련 이슈
작업 내용
DATETIME, DAYSONLY 타입에 따른 추천로직 카드 스토리북 구현
아래 표로 정리된 각 상황을 시각적으로 잘 파악하기 위해 스토리북을 작성했습니다.
🌟 변경될 요구사항 목록에 따른 추상화
위와 같이 다양한 요구사항에 대응하기 위해서는 적절하게 사용처에서 컴포넌트 분리가 필요하다고 느꼈습니다. 따라서 이를 시각적으로 보고 빠르게 정리하기 위해 아래와 같이 공통적으로 사용되는 부분을 색으로 구분하여 분리했습니다.
RecommendDateTime
: DATETIME 타입일 때 날짜를 보여줍니다.RecommendDaysOnly
: DAYSONLY 타입일 때 날짜를 보여줍니다.MeetingRecommendCard
: 주최자 외에 추천하는 시간을 보고 싶은 사용자를 위해 제공됩니다.(/meeting/:uuid/recommend)MeetingRecommendCheckboxCard
: 주최자가 보고 원하는 날짜를 선택할 수 있습니다.(/meeting/:uuid/confirm)RecommendAttendees
: “8명 중 2명”과 관련된 UI 로직이 담겨있습니다. 추후에 인원도 함께 보여주어야 하는 상황에 유연하게 대처할 수 있습니다.추천로직 30분 단위도 보여주도록 수정하기
기존 formatTime 유틸함수는 위와 같은 형태로 구현되어 있었습니다. 따라서 30분 단위는 보여주지 못했습니다.
따라서 30분 단위도 보여줄 수 있도록 수정했으며, 보다 직관적인 네이밍을 위해
formatTimeToKorean
로 유틸함수명을 수정했습니다.특이 사항
추천, 확정 페이지 컴포넌트 추상화
주황색 부분 즉,
MeetingRecommendCard
/MeetingRecommendCheckboxCard
는 역할과 책임이 잘 분리되도록 추상화했습니다. 하지만 보라색 사각형으로 보여지는 부분도 공통으로 사용되는 부분이라 추상화가 가능하지만,useMeetingTimeRecommendFilter
커스텀 훅에 의존적인 형태이기 때문에 이 부분은 추후에 추상화 해야할 것 같습니다.리뷰 요구사항 (선택)