feat: Android SDK update for version 23.1.0#120
Conversation
Greptile SummaryThis PR bumps the Android SDK to version 23.1.0, targeting Appwrite API 1.9.1. Changes include adding the Confidence Score: 5/5Safe to merge — all changes are a clean SDK version bump with no logic errors or security concerns. Only finding is a P2 suggestion to make the new userType field nullable for graceful backward compatibility; no P0/P1 issues exist. All version numbers are internally consistent (SDK, header, README, CHANGELOG), the new enum value is correctly placed, and the documentation improvements are accurate. library/src/main/java/io/appwrite/models/Log.kt — minor nullability consideration for userType Important Files Changed
Reviews (1): Last reviewed commit: "chore: update Android SDK to 23.1.0" | Re-trigger Greptile |
| userEmail = map["userEmail"] as String, | ||
| userName = map["userName"] as String, | ||
| mode = map["mode"] as String, | ||
| userType = map["userType"] as String, |
There was a problem hiding this comment.
userType nullability on older server responses
map["userType"] returns null if the key is absent, and null as String throws a kotlin.TypeCastException at runtime. Every other field in this model follows the same non-nullable pattern, so this is consistent — but userType is a newly introduced field that an Appwrite server older than 1.9.1 won't include in its response. Declaring it as String? would let apps that connect to a not-yet-upgraded backend degrade gracefully instead of crashing.
| userType = map["userType"] as String, | |
| userType = map["userType"] as? String ?: "", |
This PR contains updates to the Android SDK for version 23.1.0.