fix: do not throw NotImplemented error for existing "legacy database"file if the file is not a sqlite file

amplitude / Amplitude-Kotlin

Amplitude Kotlin SDK

MIT License

27 stars 10 forks source link

fix: do not throw NotImplemented error for existing "legacy database"file if the file is not a sqlite file #147

Closed falconandy closed 11 months ago

falconandy commented 1 year ago

Summary

Possible fix for DataMigration NotImplementedError. Data migration already checks if the Database file exists and skips the migration if it doesn't. But if the file exists but it is not a sqlite database for some reason - NotImplemented error is thrown. Now migration is skipped in both cases.

Checklist

[x] Does your PR title have the correct title format?
Does your PR have a breaking change?: No

github-actions[bot] commented 11 months ago

:tada: This PR is included in version 1.11.1 :tada:

The release is available on:

GitHub release
v1.11.1

Your semantic-release bot :package::rocket: