Skip to content

feat: SDK update for version v7.7.0 - #82

Merged
ChiragAgg5k merged 1 commit into
mainfrom
release/v7.7.0
Oct 6, 2026
Merged

ChiragAgg5k merged 1 commit into
mainfrom
release/v7.7.0

Conversation

@ChiragAgg5k

Copy link
Copy Markdown
Member

This PR contains updates to the SDK for version v7.7.0.

What's Changed

  • Added: optional state parameter on createOAuth2Token, returned unchanged on the success and failure URLs

This is a patch on top of the previous stable release: it adds only the state parameter from appwrite/appwrite#14157, plus the version bump and changelog. Generated by diffing the sdk-generator output for the specs before and after #14157.

Adds the optional state parameter to createOAuth2Token (appwrite/appwrite#14157).
@hansi-codes

hansi-codes Bot commented Oct 6, 2026

Copy link
Copy Markdown

🟢 Tier S · Ready to merge

Adds an optional OAuth2 state setter and includes its value in CreateOAuth2Token requests. Updates the SDK version headers to v7.7.0, the changelog, and the usage example.

Verdict New comments Fixed Still open
✅ Approved 0 0 0
📂 Walkthrough · 4
File Change
CHANGELOG.md Documents the optional OAuth2 state parameter for v7.7.0.
account/account.go Adds the state option, setter, and conditional request parameter.
client/client.go Updates user-agent and SDK version headers to v7.7.0.
docs/examples/account/create-o-auth-2-token.md Demonstrates the new state setter.
🔇 Filtered out · 1

Findings Hansi considered but did not post.

Finding Why
Cover the new state parameter in the account request test Verifier: The test suite has no comparable request-query assertions (the surrounding account endpoint tests also only check the HTTP method), so this is a missing-test request without an established comparable test pattern. The production change is straightforwardly covered by the shared query encoding path.

Reviewed 35a1831 · Details · Comment @hansi-codes review to re-run, or mention @hansi-codes with a question.

@hansi-codes hansi-codes Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Tier S · Looks good to merge. Summary

@ChiragAgg5k
ChiragAgg5k merged commit 4df0acf into main Oct 6, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant