xmtp / xmtp-web

XMTP web SDKs and examples, including a React SDK and quickstart example app
https://xmtp.github.io/xmtp-web/
31 stars 12 forks source link

Bug: DexieError after upgrade to latest xmtp sdks #215

Closed doktordos-eth closed 2 months ago

doktordos-eth commented 6 months ago

Describe the bug

Upgraded to latest xmtp react and js sdks and get the following errors everytime i sign in to xmtp. Screenshot 2024-03-18 085950 Screenshot 2024-03-18 090100 After manually deleting the indexed db all works as expected-

Expected behavior

No dexie errors are thrown from the sdks - schema mismatches should be handled internally without needing to manually delete the indexed db ..

Steps to reproduce the bug

upgraded from react-sdk 4.0.0 to 5.2.0 and js-sdk 11.1.1 to 11.5.0

rygine commented 6 months ago

hey @doktordos-eth! thanks for the report. it's not necessary to delete the DB, but rather just increase the schema version. sorry that this wasn't clear in our docs.

in #219, i added a schema check that will throw an error when there's a schema mismatch and inform app developers to update their schema version. hope this helps, and of course, open to feedback!

rygine commented 2 months ago

@doktordos-eth this may be resolved via #284. please re-open if the issue persists after upgrading to the latest SDK.