Skip to content

fix: 퇴사 처리 후 사장님 근무자 목록이 갱신 - #68

Merged
kim3360 merged 1 commit into
devfrom
fit/ALT-313
Oct 7, 2026
Merged

kim3360 merged 1 commit into
devfrom
fit/ALT-313

Conversation

@kim3360

@kim3360 kim3360 commented Oct 5, 2026

Copy link
Copy Markdown
Member

ID

  • ALT-313

변경 내용

  • 퇴사 처리 성공 후 사장님 근무자 목록이 재조회되지 않는 문제 수정
  • 목록 재조회가 완료될 때까지 퇴사 처리 로딩 상태 유지

구현 사항

  • 근무자 목록 무효화 키에 포함되던 불필요한 undefined 제거
  • 필터 없이 쿼리 키를 생성하면 업장별 공통 키를 반환하도록 변경
  • 상태·이름·페이지 크기에 관계없이 해당 업장의 근무자 목록 캐시 무효화
  • onSuccess에서 invalidateQueries의 Promise를 반환하여 재조회 완료 대기

구현 시연 (필요 시)

  • 작업물을 가시적으로 표현한다 (스크린샷, 영상 등)

참고 사항 (필요 시)

  • 추가적으로 검토가 필요한 사항이나 관련 문서, 참조 문서 등을 첨부
    (예: 확인 사항, 참조 링크 등)

@kim3360
kim3360 requested review from limtjdghks and oohveon October 5, 2026 16:41
@kim3360 kim3360 self-assigned this Oct 5, 2026
@vercel

vercel Bot commented Oct 5, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated
alter-client Ready Ready Preview Oct 5, 2026 4:42pm UTC

Request Review

@coderabbitai

coderabbitai Bot commented Oct 5, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 12 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: Repository: alter-app/alter-client/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 35850cc1-f1d6-4221-9efe-529931d95a87
📥 Commits

Reviewing files that changed from the base of the PR and between c7093bc and e9604b5.

📒 Files selected for processing (2)
  • src/features/manager/worker-list/hooks/mutation/useResignWorkerMutation.ts
  • src/shared/lib/queryKeys.ts
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@kim3360
kim3360 merged commit 3699fcf into dev Oct 7, 2026
4 checks passed

This branch was successfully deployed

1 active deployment
Preview — e9604b5b Deployed Oct 5, 2026 by vercel[bot]
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.

3 participants