[PR #30416] fix: correct useEducationStatus query cache configuration #32806

Closed
opened 2026-02-21 20:52:06 -05:00 by yindo · 0 comments
Owner

Original Pull Request: https://github.com/langgenius/dify/pull/30416

State: closed
Merged: Yes


Part of #30414
Fixes #30415

Summary

Replaced ineffective gcTime: 0 with proper cache controls for provider-level query.

Changes

Hook Configuration

  • Removed gcTime: 0 (ineffective for active queries)
  • Added staleTime: 0 (ensures data marked stale immediately)

Component Usage

  • Use isFetchedAfterMount to prevent displaying cached data before fresh fetch

Why

Per TanStack Query docs, gcTime only applies to inactive queries. Provider-level queries remain active, making gcTime: 0 ineffective.

Testing

  • pnpm lint:fix
  • pnpm type-check:tsgo
**Original Pull Request:** https://github.com/langgenius/dify/pull/30416 **State:** closed **Merged:** Yes --- Part of #30414 Fixes #30415 ## Summary Replaced ineffective `gcTime: 0` with proper cache controls for provider-level query. ## Changes ### Hook Configuration - ❌ Removed `gcTime: 0` (ineffective for active queries) - ✅ Added `staleTime: 0` (ensures data marked stale immediately) ### Component Usage - ✅ Use `isFetchedAfterMount` to prevent displaying cached data before fresh fetch ## Why Per [TanStack Query docs](https://tanstack.com/query/latest/docs/framework/react/reference/useQuery#gctime-number--infinity), `gcTime` only applies to **inactive** queries. Provider-level queries remain active, making `gcTime: 0` ineffective. ## Testing - ✅ pnpm lint:fix - ✅ pnpm type-check:tsgo
yindo added the pull-request label 2026-02-21 20:52:06 -05:00
yindo closed this issue 2026-02-21 20:52:07 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langgenius/dify#32806