owncloud / android

:phone: The ownCloud Android App
GNU General Public License v2.0
3.83k stars 3.05k forks source link

[2.4.0] Release new version #1957

Closed jesmrec closed 7 years ago

jesmrec commented 7 years ago

PRs: #1969


AC:


BUGS & IMPROVEMENTS

jesmrec commented 7 years ago

(1) Bug

Steps

  1. Download a txt file
  2. Open it with an external app (for example ES Note Editor)
  3. Edit and save the file

Current behaviour

File is not uploaded

Expected behaviour

File is uploaded

Server v10 (with and without SAML) Huawei 6P v7

davivel commented 7 years ago

@jesmrec , (1) is expected behaviour in the Android app right now; only av-off files should be uploaded immediately. This issue tracks the feature you are missing: https://github.com/owncloud/android/issues/1683

davigonz commented 7 years ago

@davivel @jesmrec In connection with the bug (1), I've tested it with av-off files and several apps and here are the results I've obtained:

davivel commented 7 years ago

Some editors do strange things to save files instead of just writing.

Please, test with release 2.3.0 to see if it's a real regression.

davigonz commented 7 years ago

@davivel @jesmrec tested with 2.3.0, the behavior is exactly the same as in 2.4.0 release, so is not a regression in the end

jesmrec commented 7 years ago

@davivel @davigonz i have noticed a dependency on the device and app that edit the file. There are apps/devices that can edit a file (non av. offline) and upload it to oC, but they do in 2.4.0 and 2.3.0 as well, so it would not be a bug. This is not a regression then, but we have different behaviours in any case, as dealt in https://github.com/owncloud/android/issues/1683

jesmrec commented 7 years ago

(2) Bug (fixed)

Steps

  1. Create a folder called (+-&_$éô
  2. Create two public links with default name -> OK
  3. Create another public link with default name

Current behaviour: crash Expected behaviour: public link created

Huawei6P v7 server v10.0.2 SAML

davivel commented 7 years ago

@jesmrec , (2) is ready to be tested.

I'll create PRs to check that everything is OK in Travis

jesmrec commented 7 years ago

Version approved