ARK-Builders / ARK-Navigator

Android app for navigation through your data

MIT License

15 stars 15 forks source link

#283: Condensed folder picker #318

Closed mdrlzy closed 1 year ago

mdrlzy commented 1 year ago

283

kirillt commented 1 year ago

Update erased folders, I guess that is expected since we changed path of roots list storage, but:

[x] Auto-discovery was not suggested after the update.
[x] Manual addition of a root causes RoomDB exception: http://65.21.4.137:8080/app/1/bug/95/report/93eb90c9-f755-493f-a684-12165d2cc6e2

After manual data erasure in settings menu:

[x] Auto-discovery triggers indexing of each root, which makes user wait too long.

mdrlzy commented 1 year ago

Auto-discovery was not suggested after the update.

Added rescan roots button to the settings

Manual addition of a root causes RoomDB exception

I forgot to change database version, also added small manual migration to avoid destructive migration and unnecessary indexing

After manual data erasure in settings menu: Auto-discovery triggers indexing of each root, which makes user wait too long.

You also deleted the app database, so you need to reindex roots

kirillt commented 1 year ago

You also deleted the app database, so you need to reindex roots

Yes, but we can do it lazily, waiting till particular root will be opened, isn't it? This way, user would only wait every time she opens a root for first time, but would not wait at all during opening the app.

Added rescan roots button to the settings

:+1:

I forgot to change database version, also added small manual migration to avoid destructive migration and unnecessary indexing

Will test update again.