Fix: #301 프로젝트 생성 & 할 일 생성 날짜 종료일 수정 #307
Conversation
개요Walkthrough이 변경 사항은 Changes
Sequence DiagramsequenceDiagram
participant User
participant PeriodDateInput
participant Validation
User->>PeriodDateInput: 시작 날짜 선택
alt enableEndDateSync is true
PeriodDateInput->>PeriodDateInput: 종료 날짜 자동 동기화
else enableEndDateSync is false
User->>PeriodDateInput: 종료 날짜 직접 선택
end
PeriodDateInput->>Validation: 날짜 유효성 검사
Poem
Tip CodeRabbit's docstrings feature is now available as part of our Early Access Program! Simply use the command Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (4)
src/components/common/PeriodDateInput.tsx (2)
47-55: useEffect 로직을 더 명확하게 개선할 수 있습니다.현재 조건문이 중첩되어 있어 코드의 가독성이 떨어집니다. 다음과 같이 개선해보는 것은 어떨까요?
- if (enableEndDateSync && !hasDeadline) { - setValue(endDateFieldName, null); - } else { - const startDate = startDateStr ? DateTime.fromISO(startDateStr).startOf('day') : null; - const endDate = endDateStr ? DateTime.fromISO(endDateStr).startOf('day') : null; - if (startDate && endDate) { - setHasDeadline(startDate < endDate); - } - } + if (enableEndDateSync && !hasDeadline) { + setValue(endDateFieldName, null); + return; + } + + const startDate = startDateStr ? DateTime.fromISO(startDateStr).startOf('day') : null; + const endDate = endDateStr ? DateTime.fromISO(endDateStr).startOf('day') : null; + + if (startDate && endDate) { + setHasDeadline(startDate < endDate); + }
59-69: handleDeadlineToggle 함수를 더 간단하게 리팩토링할 수 있습니다.현재 코드에서
clearErrors호출이 중복되어 있고, 조건문 구조가 복잡합니다.- if (hasDeadline === false) { - if (enableEndDateSync) { - setValue(endDateFieldName, null); - } else { - setValue(endDateFieldName, getValues(startDateFieldName)); - } - clearErrors(endDateFieldName); - } else { - setValue(endDateFieldName, getValues(startDateFieldName)); - clearErrors(endDateFieldName); - } + const newEndDate = (!hasDeadline && enableEndDateSync) + ? null + : getValues(startDateFieldName); + + setValue(endDateFieldName, newEndDate); + clearErrors(endDateFieldName);src/components/modal/task/ModalTaskForm.tsx (1)
165-165: 명시적인 값 설정을 권장합니다.기본값을 사용하는 대신
enableEndDateSync={true}로 명시적으로 설정하면 코드의 의도가 더 명확해질 것 같습니다.- enableEndDateSync + enableEndDateSync={true}src/components/modal/task/UpdateModalTask.tsx (1)
179-179: 동작 방식 차이에 대한 설명 추가를 제안합니다.작업 생성 시와 수정 시의 날짜 동기화 동작이 다른 이유를 설명하는 주석을 추가하면 좋을 것 같습니다.
+ // 작업 수정 시에는 시작일과 종료일을 독립적으로 설정할 수 있도록 동기화를 비활성화합니다. enableEndDateSync={false}
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (6)
src/components/common/PeriodDateInput.tsx(4 hunks)src/components/modal/project/ModalProjectForm.tsx(1 hunks)src/components/modal/project/UpdateModalProject.tsx(1 hunks)src/components/modal/task/ModalTaskForm.tsx(1 hunks)src/components/modal/task/UpdateModalTask.tsx(1 hunks)src/constants/formValidationRules.ts(2 hunks)
🔇 Additional comments (7)
src/components/common/PeriodDateInput.tsx (2)
17-17: 타입 정의가 명확하고 적절합니다!
enableEndDateSync 프로퍼티가 불리언 타입으로 잘 정의되어 있습니다.
31-31: 기본값 설정이 적절합니다!
enableEndDateSync의 기본값을 true로 설정한 것은 기존 동작과의 호환성을 유지하는 좋은 접근입니다.
src/constants/formValidationRules.ts (2)
11-11: 매개변수 타입 변경이 적절합니다!
referenceDate 매개변수에 null 타입을 추가한 것은 enableEndDateSync 기능과 잘 어울립니다.
202-206: 종료일 유효성 검사 로직이 잘 구현되었습니다!
enableEndDateSync 매개변수를 활용하여 종료일 유효성 검사를 조건부로 처리하는 방식이 깔끔합니다.
src/components/modal/project/ModalProjectForm.tsx (1)
160-160: PeriodDateInput 컴포넌트 사용이 적절합니다!
enableEndDateSync 프로퍼티를 추가하여 프로젝트의 날짜 동기화 동작을 명확하게 제어할 수 있게 되었습니다.
src/components/modal/project/UpdateModalProject.tsx (1)
149-149: 프로젝트 날짜 동기화 기능이 적절히 구현되었습니다.
기본값(true)을 사용하여 프로젝트의 시작일과 종료일이 자동으로 동기화되도록 설정되었습니다.
src/components/modal/task/UpdateModalTask.tsx (1)
179-179: 날짜 동기화 비활성화가 적절히 구현되었습니다.
작업 수정 시 시작일과 종료일을 독립적으로 설정할 수 있도록 구현되었습니다.
…ature/#301-project-create-date-fix
PR Type
What kind of change does this PR introduce?
Related Issues
What does this PR do?
Other information