Fix: #182 칸반 보드에서 일정 삭제 / 상태 삭제 후 순서 변경이 제대로 일어나지 않는 문제 해결 #184
Conversation
Walkthrough이 변경 사항은 Changes
Assessment against linked issues
Possibly related PRs
Suggested labels
Suggested reviewers
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
🧹 Outside diff range and nitpick comments (2)
src/mocks/services/statusServiceHandler.ts (1)
95-98: 상태 삭제 후 정렬 순서 재조정 로직 추가새로 추가된 코드는 상태 삭제 후 남은 상태들의
sortOrder를 재조정하는 기능을 구현하고 있습니다. 이는 PR의 목적과 연결된 이슈 #182에서 제기된 문제를 해결하는 데 도움이 됩니다.몇 가지 개선 사항을 제안드립니다:
- 코드의 가독성을 높이기 위해 별도의 함수로 분리하는 것이 좋겠습니다.
- 재정렬 로직에 대한 주석을 추가하여 코드의 의도를 명확히 설명하면 좋겠습니다.
- 에러 처리를 추가하여 재정렬 과정에서 발생할 수 있는 문제에 대비하는 것이 좋겠습니다.
다음과 같이 코드를 개선할 것을 제안합니다:
// 프로젝트 상태의 정렬 순서를 재조정하는 함수 const reorderProjectStatuses = (projectId: number) => { try { STATUS_DUMMY.filter((status) => status.projectId === projectId) .sort((a, b) => a.sortOrder - b.sortOrder) .forEach((status, index) => { status.sortOrder = index + 1; }); } catch (error) { console.error('상태 재정렬 중 오류 발생:', error); // 필요한 경우 추가적인 에러 처리 로직을 구현하세요 } }; // 기존 삭제 로직 이후에 호출 reorderProjectStatuses(Number(projectId));이렇게 변경하면 코드의 가독성과 유지보수성이 향상될 것입니다.
src/mocks/services/taskServiceHandler.ts (1)
239-242: LGTM! 작업 순서 재정렬 로직이 잘 구현되었습니다.삭제 후 작업 순서를 재정렬하는 로직이 효과적으로 구현되었습니다. 이는 PR의 목표와 #182 이슈를 잘 해결합니다.
성능 최적화를 위해 다음과 같이
map을 사용하여 코드를 약간 수정할 수 있습니다:- TASK_DUMMY.filter((target) => target.statusId === task.statusId) - .sort((a, b) => a.sortOrder - b.sortOrder) - .forEach((task, index) => (task.sortOrder = index + 1)); + TASK_DUMMY.filter((target) => target.statusId === task.statusId) + .sort((a, b) => a.sortOrder - b.sortOrder) + .map((task, index) => ({ ...task, sortOrder: index + 1 }));이 변경은 기존 객체를 직접 수정하는 대신 새로운 객체를 생성하여 불변성을 유지합니다.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
📒 Files selected for processing (2)
- src/mocks/services/statusServiceHandler.ts (1 hunks)
- src/mocks/services/taskServiceHandler.ts (1 hunks)
🔇 Additional comments (1)
src/mocks/services/statusServiceHandler.ts (1)
Line range hint
1-102: 전체 변경 사항 요약 및 영향 평가이 PR은 칸반 보드에서 상태 삭제 후 정렬 순서가 제대로 업데이트되지 않는 문제를 해결하고 있습니다. 주요 변경 사항은 다음과 같습니다:
- 상태 삭제 API 핸들러에 정렬 순서 재조정 로직이 추가되었습니다.
- 삭제 후 남은 상태들의
sortOrder가 올바르게 업데이트됩니다.이러한 변경으로 인한 영향:
- 사용자 경험이 개선되어 칸반 보드의 일관성이 유지됩니다.
- 새로운 상태 추가 시 정렬 순서 충돌 문제가 해결됩니다.
추가 고려 사항:
- 다른 API 핸들러(생성, 수정)에서도 유사한 정렬 로직이 필요한지 검토해 보시기 바랍니다.
- 실제 백엔드 구현 시 이 로직이 동일하게 적용되어야 함을 명심하세요.
- 프론트엔드에서 이 변경사항을 활용할 수 있도록 관련 컴포넌트나 상태 관리 로직을 업데이트해야 할 수 있습니다.
전반적으로 이 변경사항은 문제를 효과적으로 해결하고 있으며, 제안된 개선사항을 적용하면 코드 품질이 더욱 향상될 것입니다.
Seok93
changed the title
|
@ice-bear98 @Yoonyesol 리뷰 감사합니다 ㅎㅎ |
PR Type
What kind of change does this PR introduce?
Related Issues
What does this PR do?