feat: React Native SDK update for version 0.30.0#104
feat: React Native SDK update for version 0.30.0#104ArnabChatterjee20k wants to merge 3 commits into
Conversation
Greptile SummaryThis PR updates the React Native SDK to version 0.30.0, adding
Confidence Score: 5/5The PR is safe to merge. All core logic changes are well-guarded, the GitHub Actions SHA pins are verified against official releases, and the new services follow established SDK patterns. The new Realtime presence flow is carefully implemented with a single-flight socket lock and an application-level connected gate. The REST services are straightforward GET/PUT/PATCH/DELETE wrappers. The only gap is a missing No files require special attention, though Important Files Changed
Reviews (2): Last reviewed commit: "chore: update React Native SDK to 0.30.0" | Re-trigger Greptile |
| import * as FileSystem from 'expo-file-system'; | ||
| import { Platform as RNPlatform } from 'react-native'; |
There was a problem hiding this comment.
Neither
FileSystem nor RNPlatform are referenced anywhere in Advisor — it contains only GET endpoints with no file I/O. These appear to be copy-paste artifacts from upload-capable services (e.g. Storage). Bundlers that don't tree-shake may unnecessarily include the expo-file-system module.
| import * as FileSystem from 'expo-file-system'; | |
| import { Platform as RNPlatform } from 'react-native'; |
| import * as FileSystem from 'expo-file-system'; | ||
| import { Platform as RNPlatform } from 'react-native'; |
There was a problem hiding this comment.
| status: '<STATUS>', // optional | ||
| expiresAt: '2020-10-15T06:38:00.000+00:00', // optional | ||
| metadata: {}, // optional | ||
| permissions: ["read("any")"], // optional |
There was a problem hiding this comment.
The permission string
"read("any")" is a JavaScript syntax error. The inner double-quotes terminate the outer string literal before the closing ), so pasting this snippet will throw a parse error. The file already imports Permission and Role, so the idiomatic form should be used instead.
| permissions: ["read("any")"], // optional | |
| permissions: [Permission.read(Role.any())], // optional |
| const result = await presences.upsert({ | ||
| presenceId: '<PRESENCE_ID>', | ||
| status: '<STATUS>', | ||
| permissions: ["read("any")"], // optional |
There was a problem hiding this comment.
Same invalid permission string as in
update.md — "read("any")" is a JavaScript syntax error. The imported Permission and Role helpers should be used here too for consistency and correctness.
| permissions: ["read("any")"], // optional | |
| permissions: [Permission.read(Role.any())], // optional |
This PR contains updates to the React Native SDK for version 0.30.0.
What's Changed
presenceschannel andRealtimePresencetypes for presence subscriptionsAdvisorandPresencesservicesInsight,Presence, andReportmodels with list variantsfusionauth,keycloak, andkickproviders toOAuthProviderenumClient.setCookie()method for forwarding cookies in server-side runtimesX-Appwrite-Response-Formatheader to1.9.5