Bugfixes

[l-jonas]

@Nutomic I created an release build (which contains this) and verified that it works. For the releasing, I would prefer if someone else (e.g. you) would test it too because broken builds are very bad (this applies to all releases, not just this one).

[Nutomic]

Okay some things I noticed:

• The dialog "Another Syncthing instance is running" pops up every time you click on a folder, and every time you go back to the main Activity. Ideally it should only show up once on app start, and maybe have a checkbox/setting "don't show this again".
• The string dialog_warning_reconnect_problem is a bit unclear, instead of "connections over wifi" you should say something like "local discovery connections" or "local connections". And maybe try to shorten the entire message.
• If the remote device is disconnected, you can still open a folder, and try to download a file. The app shows a "Downloading" dialog and then fails after a few seconds. That should be handled earlier (but not necessarily in this release).
• Connecting to a remote device was pretty slow to me and ifle downloads failed, but that's probably because my internet connection is really bad, and the test server is really far away.

[l-jonas]

@Nutomic

To the "another syncthing instance is running" warning:

This is not good. I created an issue for it at #62.

To the reconnect warning:

I used the term WiFi because most people understand it better than "local network" or "local connections". But I'm not sure about the audience of syncthing lite.

To the server offline handling:

It would be possible to save that one connecting attempt has been made already to fail early (without trying to connect again). And the error message could be better when the server was not reached. I created an issue at #63

To the failing downloading:

• Does it work if you increase the timeout at

  syncthing-lite/syncthing-bep/src/main/kotlin/net/syncthing/java/bep/BlockPuller.kt

  Line 99 in 9e632c2

  val blockContent = pullBlock(fileBlocks, block, 1000 * 15 /* 15 seconds timeout per block */)

  ?
• Could you create an issue for it (with a log)?

[Nutomic]

I think the error is misleading, because when I connect to my remote server, it is also a "connection over wifi". And I think it makes sense if we use terms that are consistent with Syncthing.

And I will give that a try later :)

[Nutomic]

Okay it works fine after increasing the timeout to 60 seconds 👍

By the way, seeing x/1000 looks weird, I would really go with x/100.

[l-jonas]

The things were changed (except the server offline handling as this is more complex)

[Nutomic]

Much better, thanks!

Now you can just make a commit to master with the new versionName and versionCode, and tag it.