-
Notifications
You must be signed in to change notification settings - Fork 117
Mention the need to possibly upgrade SDKs #216
base: main
Are you sure you want to change the base?
Conversation
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 :) PS: I'm putting the actual error message here so google can find it.
|
app/views/docs/upgrade.phtml
Outdated
<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> |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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.
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. |
Also, ideally, there shouldn't be a need to upgrade your SDKs right away. They should be backwards compatible (at least for 1 version). |
No description provided.