-
Notifications
You must be signed in to change notification settings - Fork 4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
🐛 [cloud_firestore] 4.15.0 notSetQueryParam is invalid type and breaks our build #12240
Comments
@danielmahon |
I can confirm this bug. When using cloud_firestore 4.15.0 and cloud_firestore_odm. Example: ` part 'rc_customer_record.g.dart'; @JsonSerializable() String? purchase_date; factory OtherPurchase.fromJson(Map<String, dynamic> json) => @JsonSerializable() String? expires_date; factory Entitlement.fromJson(Map<String, dynamic> json) => Map<String, dynamic> toJson() => _$EntitlementToJson(this); @JsonSerializable() String? auto_resume_date; factory Subscription.fromJson(Map<String, dynamic> json) => Map<String, dynamic> toJson() => _$SubscriptionToJson(this); @JsonSerializable() String? id; factory NonSubscription.fromJson(Map<String, dynamic> json) => Map<String, dynamic> toJson() => _$NonSubscriptionToJson(this); const firestoreSerializable = JsonSerializable( @Collection('rc_customers') factory RcCustomerRecord.fromJson(Map<String, dynamic> json) => Map<String, dynamic> toJson() => _$RcCustomerRecordToJson(this); final rc_customerRef = RcCustomerRecordCollectionReference(); produces a generated *.g file.
With cloud_firestore v4.15.0 notSetQueryParam is an invalid type. The workaround of locking cloud_firestore v4.14.0 works. |
Any solution for this? |
Caused by this change cff6f76 |
I forgot to publish the fix for this. Fixed in -dev.83 And yes, issues should be reported to the new repo now. |
This seems to be have broken querying capabilities FirebaseExtended/firestoreodm-flutter#7 |
Bug report
After upgrading to 4.15.0
notSetQueryParam
can no longer be accessed by ourmodels.g.dart
file. It returns an invalid type and breaks the build. Locking version to 4.14.0 fixes the issue.Steps to reproduce
Expected behavior
notSetQueryParam
is accessible from the generated fileAdditional Info
Flutter doctor
Run
flutter doctor
and paste the output below:Click To Expand
Flutter dependencies
Run
flutter pub deps -- --style=compact
and paste the output below:Click To Expand
The text was updated successfully, but these errors were encountered: