-
-
Notifications
You must be signed in to change notification settings - Fork 3.4k
Closed
Description
The query filters have lots of flags, some of them contradicting each other, and it become quite confusing very fast and it also supports invalid combinations. Especially the active
and inactive
resp. refetchActive
and refetchInactive
flags are problematic, as outlined in this issue
Proposed solution
Replace the active
and inactive
booleans with a QueryStatusFilter: 'all' | 'active' | 'inactive'
. We already map the booleans internally (see here), but we currently also allow for a 'none'
state if both booleans are set to false
, which doesn't make any sense - you wouldn't mach any query at all.
lubieowoce