Repository navigation
feat: Ruby SDK update for version 29.0.0 - #74
ChiragAgg5k wants to merge 2 commits into
Conversation
馃數 Tier A 路 Mergeable after minor fixes
Updates the Ruby SDK to version 29.0.0 and Appwrite response format 2.4.0. It adds passkey, domains, WAF, OAuth, policy, and messaging APIs and models, adjusts client request/response handling, and refreshes generated examples and documentation. 340 of 563 changed files were too large to include in full.
Note Part of the diff was too large to review, so Hansi did not approve.
Fix with agent prompt### Issue 1
docs/examples/graphql/mutation.md:15
**Fix the Ruby quoting in the mutation example**
The inner `"Walter"` closes the outer string, so this example has a Ruby syntax error when copied. Use single quotes around the GraphQL document or escape the inner quotes.
### Issue 2
docs/examples/domains/get-price.md:16
**Use Ruby `nil` for the optional period**
Ruby has no `null` literal, so this example raises an error before making the request; use `nil` for the optional `period_years` value. The `domains/list-prices.md` example has the same issue.
---
For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.馃搨 Walkthrough 路 6
Reviewed |
| result = graphql.mutation( | ||
| query: {} | ||
| query: { | ||
| "query" => "mutation { accountUpdateName(name: "Walter") { name } }" |
There was a problem hiding this comment.
Fix the Ruby quoting in the mutation example
The inner "Walter" closes the outer string, so this example has a Ruby syntax error when copied. Use single quotes around the GraphQL document or escape the inner quotes.
Prompt To Fix With AI
This is a comment left during a code review.
Path: docs/examples/graphql/mutation.md
Line: 15
Comment:
**Fix the Ruby quoting in the mutation example**
The inner `"Walter"` closes the outer string, so this example has a Ruby syntax error when copied. Use single quotes around the GraphQL document or escape the inner quotes.
---
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.
|
|
||
| result = domains.get_price( | ||
| domain: 'example.com', | ||
| period_years: null, # optional |
There was a problem hiding this comment.
Use Ruby nil for the optional period
Ruby has no null literal, so this example raises an error before making the request; use nil for the optional period_years value. The domains/list-prices.md example has the same issue.
Prompt To Fix With AI
This is a comment left during a code review.
Path: docs/examples/domains/get-price.md
Line: 16
Comment:
**Use Ruby `nil` for the optional period**
Ruby has no `null` literal, so this example raises an error before making the request; use `nil` for the optional `period_years` value. The `domains/list-prices.md` example has the same issue.
---
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.
|
Superseded: replaced by a passkeys-only minor release. |
This PR contains updates to the Ruby SDK for version 29.0.0.