oldergod / android-architecture

MVI architecture Implementation of the ToDo app.
Apache License 2.0
669 stars 70 forks source link

Crash on add TODO item #39

Closed tellypresence closed 6 years ago

tellypresence commented 6 years ago

Easily reproduced:

oldergod commented 6 years ago

Hey, thank you for the report. Was branch were you testing on? I don't think we have any file packaged under com.example.android.architecture.blueprints.todomvp.mock.

Are you sure it's not one of the branch of the parent project here? https://github.com/googlesamples/android-architecture

tellypresence commented 6 years ago

Apologies, your app is not at fault. The problem was that I had a different variant (might have been mvvm-rx or mvp-coroutines etc) which had a slightly different DB schema, so when I installed your app over that one it crashed.

Simply uninstalling the apk, then relaunching your app works perfectly.

(Fix could be as simple as appending e.g. ".mvi" to android.defaultConfig.applicationId)