Skip to content

Conversation

@ptyoiy
Copy link
Contributor

@ptyoiy ptyoiy commented Dec 29, 2025

✅ Done Task

  • 시간 계산 로직 수정
  • 누락된 queryKey 컨벤션 적용 (문제를 등록해도 새로고침해야 화면에 나오던 버그)
    • queryKey 컨벤션 적용에 따라 페이지네이션 searchParam 추가

☀️ New-insight

💎 PR Point

  • 시간 계산 로직
    기존의 시간 계산 로직은 startDateDate.now()와 비교했었는데, 문제 등록 폼이 시간을 받지 않기 때문에 Date.now()와 비교해서는 안됐습니다. 날짜까지만 입력받는 문제 등록 폼의 맥락에 맞게 date-fnsisBefore로 내일 날짜와 비교하도록 수정했습니다. (12/29 기준, startDate >= 12/30면 대기중인 문제 탭으로 이동)

  • queryKey 컨벤션 누락
    query 부분은 ["queuedProblem", groupId] 와 같이 옛날 방식으로 작성된 queryKey를 사용하고 있는 반면, mutation에서는 queryKey 컨벤션인 groupQueryKey를 사용하고 있습니다. 이에 따른 둘 간의 불일치로 인해 문제를 등록해도 invalidate하지 못하는 일이 발생하고 있었습니다. 그래서 잘 수정했습니다.

  • 페이지네이션 searchParam 적용
    usePaginationQueryqueryKey 컨벤션을 적용함에 따라 페이지네이션 용 searchParamgroupQueryKey의 0번 인덱스인 group으로 적용되므로 각각에 맞는 searchParam을 따로 적용해 주었습니다.

📸 Screenshot

@ptyoiy ptyoiy self-assigned this Dec 29, 2025
@ptyoiy ptyoiy added the 🐛 Fix 버그 수정 및 개선 label Dec 29, 2025
@linear
Copy link

linear bot commented Dec 29, 2025

@channeltalk
Copy link

channeltalk bot commented Dec 29, 2025

@github-actions
Copy link

🚀 CI Check Results

Check Item Status
🔷 TypeScript ✅ Pass
🔍 ESLint ✅ Pass
🎨 Format ❌ Fail

📋 View Full Workflow

@github-actions
Copy link

🚀 CI Check Results

Check Item Status
🔷 TypeScript ✅ Pass
🔍 ESLint ✅ Pass
🎨 Format ✅ Pass

📋 View Full Workflow

Copy link
Member

@j-nary j-nary left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 고생하셨습니당 ~ 👍

Copy link
Contributor

@rtttr1 rtttr1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니다!!

@ptyoiy ptyoiy merged commit 93f58af into main Jan 1, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛 Fix 버그 수정 및 개선 size/s

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants