hotfix: 오늘을 선택한경우 지금 시간보다 이전의 시간 옵션을 disabled 처리#841
Closed
spoyodevelop wants to merge 0 commit intofe/devfrom
Closed
hotfix: 오늘을 선택한경우 지금 시간보다 이전의 시간 옵션을 disabled 처리#841spoyodevelop wants to merge 0 commit intofe/devfrom
spoyodevelop wants to merge 0 commit intofe/devfrom
Conversation
spoyodevelop
requested review from
Db0111,
hoyyChoi,
jhan0121,
m-a-king and
thgml05
⚡️ Lighthouse Report
📊 Performance Details (성능 세부 지표)
|
spoyodevelop
commented
Mar 12, 2026
| import { useMemo } from 'react'; | ||
| import { useRoomSelector } from '../store/createRoomStore'; | ||
|
|
||
| export interface HourOption { |
Contributor
Author
There was a problem hiding this comment.
HourOption이라는 새로운 Type이 추가 되었습니다. 해당부분은 임시적인 fix이며, 후에 서버측 validation(v3)가 추가될시, 롤백 가능합니다.
|
🚀 Storybook preview: https://687852cd7789368357f3bb71-gvlhipuisj.chromatic.com/ |
spoyodevelop
force-pushed
the
fe/fix/past-time
branch
from
0995a70 to
722dc72
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
#️⃣ 연관된 이슈
서버측 validation로직이 변경되면서, 이전 날짜/시간 validation을 client측에서 처리하는 로직을 추가했습니다.
📝 작업 내용
CleanShot.2026-03-12.at.20.17.27.mp4
오늘을 선택하면 지금보다 이전 시간대를 disabled 처리하고, 오늘을 선택하지 않으면 disabled를 풉니다.
일반적인 유저 사용법으로는 에러를 유도하기 힘들고, 간단한 fix로는 충분한것 같아서 해당 PR을 작성했습니다.
현재 구조상 오늘을 선택하면, 그 이후의 시간대를 선택해도 무조건 지금의 시간대 (예를들면 3시를 선택한경우 4시부터만 선택을 가능하게 할수 있음)만 선택이 가능하네요. 아이고.... 가만히 생각하니 그렇네요.... ㅋㅋㅋㅋㅋㅋ
💬 리뷰 요구사항
해당과정중에서 불가피하게 TimePicker 컴포넌트에 수정을 가했습니다. 검증로직이 간단해지는 v3가 포함되면 PR의 변경사항은 롤백도 고려해 보면 좋을것 같습니다.