Repository navigation
feat: API specs update for version 2.4.x - #125
Conversation
🔵 Tier A · Mergeable after minor fixes
This PR adds the 2.4.x API specification and generated SDK example set, covering updated endpoints across client and server SDKs. The new examples include account, apps, databases, avatars, OAuth2, and other API groups. 15519 of 15665 changed files were too large to include in full.
Note @appwrite-specifications[bot] does not have write access to this repository, so Hansi does not approve automatically. A maintainer can review and approve.
Fix with agent prompt### Issue 1
examples/2.4.x/client-android/java/account/list-consents.md:15
**Import `java.util.List` in the Java example**
This snippet uses `List.of()` but its imports omit `java.util.List` (and other Java examples also use `Map` without importing it). Copying the example into a Java source file as written fails to compile with an unresolved `List` symbol.
### Issue 2
examples/2.4.x/client-android/java/avatars/get-screenshot.md:35
**Pass screenshot permissions as a list**
The OpenAPI parameter `permissions` is an array of `BrowserPermission` values (see `specs/2.4.x/open-api3-2.4.x.json` around line 17682), but this example passes a single enum value. The Android Java example therefore does not match the request shape; the corresponding Kotlin example has the same scalar argument.
---
For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.📂 Walkthrough · 2
Reviewed |
| Account account = new Account(client); | ||
|
|
||
| account.listConsents( | ||
| List.of(), // queries (optional) |
There was a problem hiding this comment.
Import java.util.List in the Java example
This snippet uses List.of() but its imports omit java.util.List (and other Java examples also use Map without importing it). Copying the example into a Java source file as written fails to compile with an unresolved List symbol.
Prompt To Fix With AI
This is a comment left during a code review.
Path: examples/2.4.x/client-android/java/account/list-consents.md
Line: 15
Comment:
**Import `java.util.List` in the Java example**
This snippet uses `List.of()` but its imports omit `java.util.List` (and other Java examples also use `Map` without importing it). Copying the example into a Java source file as written fails to compile with an unresolved `List` symbol.
---
For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.🟡 Minor · documentation · Reply if this doesn't apply.
| -122.4194, // longitude (optional) | ||
| 100.0, // accuracy (optional) | ||
| true, // touch (optional) | ||
| BrowserPermission.GEOLOCATION, // permissions (optional) |
There was a problem hiding this comment.
Pass screenshot permissions as a list
The OpenAPI parameter permissions is an array of BrowserPermission values (see specs/2.4.x/open-api3-2.4.x.json around line 17682), but this example passes a single enum value. The Android Java example therefore does not match the request shape; the corresponding Kotlin example has the same scalar argument.
Prompt To Fix With AI
This is a comment left during a code review.
Path: examples/2.4.x/client-android/java/avatars/get-screenshot.md
Line: 35
Comment:
**Pass screenshot permissions as a list**
The OpenAPI parameter `permissions` is an array of `BrowserPermission` values (see `specs/2.4.x/open-api3-2.4.x.json` around line 17682), but this example passes a single enum value. The Android Java example therefore does not match the request shape; the corresponding Kotlin example has the same scalar argument.
---
For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.🟡 Minor · documentation · Reply if this doesn't apply.
This PR contains API specification updates for version 2.4.x.