Skip to content

fix: SDK update for version 30.2.1 - #89

Merged
ChiragAgg5k merged 2 commits into
mainfrom
release/30.2.1
Oct 12, 2026
Merged

ChiragAgg5k merged 2 commits into
mainfrom
release/30.2.1

Conversation

@ChiragAgg5k

@ChiragAgg5k ChiragAgg5k commented Oct 12, 2026 •

Copy link
Copy Markdown
Member

This PR contains updates to the SDK for version 30.2.1.

What's Changed

  • Fixed: requires utopia-php/client ^0.6 and imports Utopia\Client\Client, so the SDK installs and runs next to client 0.6, which dropped the Utopia\Client alias that 30.2.0 imports

The generator already has this: e46f3def (released in 5.3.5) and cbcc36c7 (released in 5.5.3), both in 5.6.0. 30.2.0 missed it because it was built by applying only the interval diff onto the 30.1.0 tag, which predates those template changes. It's needed so utopia-php/migration can require 30.2 for function intervals: migration requires client ^0.6, and 30.2.0 pins ^0.4.1.

The SDK's 870 unit tests pass against client 0.6.0.

Client 0.6 dropped the Utopia\Client alias that 30.2.0 imports, so 30.2.0
cannot be installed next to client 0.6 and fails at runtime where another
package provides it.
@hansi-codes

hansi-codes Bot commented Oct 12, 2026

Copy link
Copy Markdown

🟢 Tier S · Ready to merge

The dependency constraint and updated imports consistently match the client 0.6 API.

Updates the SDK to require utopia-php/client 0.6 and use its Utopia\Client\Client class. The changelog and SDK version headers now identify release 30.2.1, and the client test uses the updated namespace.

Verdict New comments Fixed Still open
✅ Approved 0 0 0
📂 Walkthrough · 4
File Change
CHANGELOG.md Documents the 30.2.1 dependency and namespace update.
composer.json Requires utopia-php/client ^0.6.
src/Appwrite/Client.php Imports the 0.6 client class and updates SDK version headers to 30.2.1.
tests/Appwrite/ClientTest.php Uses the updated client class namespace in tests.

Reviewed a0f62ef · 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 1aace0d into main Oct 12, 2026
2 checks passed
@ChiragAgg5k
ChiragAgg5k deleted the release/30.2.1 branch October 12, 2026 09:43
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