Skip to content
This repository has been archived by the owner on Sep 29, 2023. It is now read-only.

Mention the need to possibly upgrade SDKs #216

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

zwarag
Copy link

@zwarag zwarag commented Jun 30, 2022

No description provided.

@zwarag
Copy link
Author

zwarag commented Jun 30, 2022

Hi Appwrite Team. After the upgrade from 0.14.2 to 0.15.1, I got a strange error in my devconsole [1] and got stuck a little. Drake helped me out and with his help I figured that I just needed to upgrade the SDK, as I still was using the npm version 8.0.1. So as a little thank you, I created this PR. It's not much but, maybe a little help for other users :)

[1] image

PS: I'm putting the actual error message here so google can find it.

ERROR AppwriteException: User (role: member) missing scope (collections.read)

<h3><a href="/docs/upgrade#upgrade-sdk" id="upgrade-sdk">Upgrade SDK</a></h3>

<p>After upgrading the Appwrite server, ensure that the SDKs used by your applications are compatible with the new Appwrite server version.</p>
Copy link
Member

Choose a reason for hiding this comment

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

@gewenyu99 maybe we should also add a hint on how to check if the SDK is compatible?

Copy link
Contributor

@gewenyu99 gewenyu99 Jul 15, 2022

Choose a reason for hiding this comment

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

Hmmm, yes.
@zwarag let's add a line that suggests the user to check their SDK version. I think we should suggest they upgrade the SDKs to the latest version if the user is migrating to the latest version. They should check the change log for each SDK on Github for version-specific compatibility.

For example https://github.com/appwrite/sdk-for-web/releases

Copy link
Author

Choose a reason for hiding this comment

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

@gewenyu99 sure, Thx for the feedback. I'll update the PR soon.

Copy link
Author

Choose a reason for hiding this comment

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

@gewenyu99 I've updated the commit. If there is anything I can do to improve it. Let me know.

@stnguyen90
Copy link
Contributor

ERROR AppwriteException: User (role: member) missing scope (collections.read)

Actually, this particular error was due to a bug in Appwrite which should be resolved as of Appwrite version 0.15.2.

There's also another bug you'll face if you haven't upgraded your SDK. The fix for this bug is currently waiting for release.

@stnguyen90
Copy link
Contributor

Also, ideally, there shouldn't be a need to upgrade your SDKs right away. They should be backwards compatible (at least for 1 version).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants