Skip to content

fix: 다가오는 일정 가져오는 메서드에서 undefined 가능성 오류 처리#63

Merged
GamjaIsMine02 merged 1 commit into
devfrom
feature/plan-management-page
Jan 31, 2026
Merged

fix: 다가오는 일정 가져오는 메서드에서 undefined 가능성 오류 처리#63
GamjaIsMine02 merged 1 commit into
devfrom
feature/plan-management-page

Conversation

@GamjaIsMine02
Copy link
Copy Markdown
Contributor

@GamjaIsMine02 GamjaIsMine02 commented Jan 31, 2026

1) 작업한 이슈번호

2) 변경 요약 (What & Why)

  • 무엇을 변경했는지:
  • 변경했는지(문제/목표):

3) 스크린샷/동영상 (UI 변경 시)

전/후 비교, 반응형(모바일/데스크톱) 캡쳐

  • Before:
  • After:

4) 상세 변경사항

  • 라우팅/페이지:
  • 컴포넌트:
  • 상태관리:
  • API 호출:
  • 스타일:
  • 기타:

5) 참고사항

Summary by CodeRabbit

버그 수정

  • Bug Fixes
    • 계획 항목의 정렬 로직이 개선되어 마감일이 없는 항목도 올바르게 정렬됩니다.
    • 계획 항목이 마감일 순서로 정렬되며, 마감일이 동일한 경우 생성일 순서로 정렬됩니다.

✏️ Tip: You can customize this high-level summary in your review settings.

@vercel
Copy link
Copy Markdown

vercel Bot commented Jan 31, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
devflow Building Building Preview, Comment Jan 31, 2026 5:50pm

@GamjaIsMine02 GamjaIsMine02 merged commit 4414d0f into dev Jan 31, 2026
1 of 3 checks passed
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Jan 31, 2026

Caution

Review failed

The pull request is closed.

📝 Walkthrough

개요

fetchUpcomingPlanItems 함수의 정렬 로직을 재작성하여 deadline이 없는 경우를 명시적으로 처리하고, 두 단계 정렬(deadline 기준 후 createdAt 기준)로 개선하였습니다.

변경 사항

Cohort / File(s) 요약
정렬 로직 개선
services/plans/planManageService.service.ts
fetchUpcomingPlanItems의 정렬 메커니즘을 리워크하여 deadline 부재 처리를 강화했습니다. getTime() 또는 Infinity를 통한 명시적 deadlineTime 할당과 deadlineTime(주 정렬) 및 createdAt(보조 정렬)을 기반한 이중 정렬 체계를 도입했습니다.

예상 코드 리뷰 난이도

🎯 2 (Simple) | ⏱️ ~10분

🐰 정렬의 길을 다시 다지네,
deadline 없어도 흔들리지 않고,
두 계층으로 순서 맞추며,
무한대로 안정성 지켜낸다네! ✨

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feature/plan-management-page

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant