Skip to content
This repository has been archived by the owner on Jun 19, 2023. It is now read-only.

fix the warning in main thread #1061

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

albertChien-tw
Copy link

@albertChien-tw albertChien-tw commented Jun 6, 2018

while sync favorites files is failure sometimes beacase not in the main thread


BUGS & IMPROVEMENTS

@CLAassistant
Copy link

CLAassistant commented Jun 6, 2018

CLA assistant check
All committers have signed the CLA.

@jesmrec
Copy link
Contributor

jesmrec commented Jul 10, 2018

Thanks for the contribution! to be included in next 3.8.0 version.

In any case, we are working in a new project: https://github.com/owncloud/ios-app in which you can contribute as well!!

@jesmrec
Copy link
Contributor

jesmrec commented Aug 2, 2018

i will start a QA stage to the multiselection feature.

@jesmrec
Copy link
Contributor

jesmrec commented Aug 2, 2018

BUG (1)

Steps

  1. Tap on edit
  2. Select some files
  3. Tap on Done
  4. Select "Delete" (no matter if local or remote)

Current behaviour: Only is deleted the first one (locally)
Expected: All selected files are deleted locally

Tested with: iOS11.4

@jesmrec
Copy link
Contributor

jesmrec commented Aug 2, 2018

BUG (2)

Steps

  1. Tap on Edit
  2. Tap on Done with no selection

Current: Sheet with options displayed
Expected: no item was selected, so the options should not be displayed

Tested with: iOS11.4

@jesmrec
Copy link
Contributor

jesmrec commented Aug 2, 2018

BUG (3)

Steps

  1. Tap on Edit
  2. Select items
  3. Tap on Done
  4. Tap on Cancel (edit mode is closed)
  5. Tap again on Edit

Current: Previous selection remains
Expected: Previous selections should be cleared after "Done"

Tested with: iOS11.4

@jesmrec
Copy link
Contributor

jesmrec commented Aug 2, 2018

Why is the file list moved to the right when you tap on "Edit"? i guess it is for showing the multiselection mode. The point is that blank at the left should be placed with any indicator or similar.

@jesmrec
Copy link
Contributor

jesmrec commented Aug 2, 2018

@dabechien will you check the bugs & improvements? in order to continue the process to improve the PR quality.

@albertChien-tw
Copy link
Author

albertChien-tw commented Aug 8, 2018 via email

@jesmrec
Copy link
Contributor

jesmrec commented Aug 8, 2018

ok, thanks for your engagement. When you come back, ping us, to move forward the stuff.

do not forget that we are working in a new iOS project:

https://github.com/owncloud/ios-app
https://github.com/owncloud/ios-sdk

it would be cool you to take part there!!

@jesmrec jesmrec modified the milestones: 3.8.0, backlog Aug 10, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants