Description
The current type of the value for the isFilter is bool? which does not allow specifying unknown even though that is supported by postgrest: https://docs.postgrest.org/en/v12/references/api/tables_views.html#operators
|
PostgrestFilterBuilder<T> isFilter(String column, bool? value) { |
Suggested solution
I propose to add a new enum with four values for this. While bool? is a great type for the other values: null, true, false it does now work for the remaining one unknown. This also prevents from accidentally using null.
This would be a breaking change for v3.
Alternative
No response
Additional context
No response
Validations
Description
The current type of the value for the
isFilterisbool?which does not allow specifyingunknowneven though that is supported by postgrest: https://docs.postgrest.org/en/v12/references/api/tables_views.html#operatorssupabase-flutter/packages/postgrest/lib/src/postgrest_filter_builder.dart
Line 227 in 6e11db6
Suggested solution
I propose to add a new enum with four values for this. While
bool?is a great type for the other values:null,true,falseit does now work for the remaining oneunknown. This also prevents from accidentally usingnull.This would be a breaking change for v3.
Alternative
No response
Additional context
No response
Validations