Conversation
Greptile SummaryThis PR bumps the Flutter SDK to version 24.1.1, removing the admin-only
Confidence Score: 4/5Safe to merge; the Advisor removal is clean and the version bump is consistent, but The Advisor removal is thorough — all files, exports, docs, and tests are deleted with no dangling references. The version bump is consistent across all four files that carry it. The only concern is lib/src/models/file.dart — the new Important Files Changed
Reviews (2): Last reviewed commit: "Commit from GitHub Actions (Format and p..." | Re-trigger Greptile |
| signature: map['signature'].toString(), | ||
| mimeType: map['mimeType'].toString(), | ||
| sizeOriginal: map['sizeOriginal'], | ||
| sizeActual: map['sizeActual'], |
There was a problem hiding this comment.
sizeActual is a new required field but is parsed without a null-safe fallback. If an Appwrite server omits this key (e.g., a backend version that predates this field), map['sizeActual'] returns null, which gets passed to a non-nullable int, throwing a Null check operator used on a null value at runtime on every storage file fetch. Adding a ?? 0 default (matching the pattern used for $permissions) prevents this crash.
| sizeActual: map['sizeActual'], | |
| sizeActual: map['sizeActual'] ?? 0, |
This PR contains updates to the Flutter SDK for version 24.1.1.
What's Changed
Advisorservice andInsight,InsightCTA,InsightList,Report,ReportListmodels (admin-only endpoints, not intended for client SDKs)sizeActualfield toFilemodel