Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use notifcation bar to show upload status #725

Open
Bullnados opened this issue Mar 6, 2017 · 15 comments
Open

Use notifcation bar to show upload status #725

Bullnados opened this issue Mar 6, 2017 · 15 comments
Labels
enhancement feature: transfers [WIP] Any uploading / downloading matters that aren't auto upload or sync specific hotspot: uploads tab and transfer notifications The uploads tab and upload + download related notifications needs review Potentially out of date or duplicate and/or fixed or mostly fixed so needs a status update

Comments

@Bullnados
Copy link

Would be nice to see upload status in notification bar.

Infos could be:

  • startime
  • endtime
  • upload rate in mb/s
  • erros during upload

With this also drop support for 4.0 and use "new" API 16. So it is possible to pause and restart upload over notification bar. You wont lose much devices and about 97,4% still compatible with API 16

@Bullnados
Copy link
Author

Is that still implemented? With 1.4.2 I see Nexcloud icon during upload.

@AndyScherzinger
Copy link
Member

AndyScherzinger commented Mar 14, 2017

Is that still implemented?

No has started working on this to my knowledge

With this also drop support for 4.0 and use "new" API 16. So it is possible to pause and restart upload over notification bar. You wont lose much devices and about 97,4% still compatible with API 16

We can implement such a mechanism via the notification for v16+ so we don't have to drop support for 4.0 while implement features for newer Android versions nonetheless :)

@AndyScherzinger
Copy link
Member

AndyScherzinger commented Mar 14, 2017

What do you think @tobiasKaminsky @mario? This would be a complete rewrite of the notifications since the are per file at the moment and there is another issue which asked for pause/abort on a per file level see #193

@mario
Copy link
Contributor

mario commented Mar 14, 2017

@AndyScherzinger I am actually against notifications entirely and would prefer to just move all upload/download/sync work to background jobs rather than services that we use right now.

@AndyScherzinger
Copy link
Member

@mario would upload/download/sync work still be that or would it be considered an activity/notification in the Nextcloud notification/activity sense?

@mario
Copy link
Contributor

mario commented Mar 14, 2017

Since the plan is to eliminate the separate Upload screen, Uploads/Syncs/Downloads/Whatever would be part of the activity, yes. I'd argue against the ability to pause manually, but would like to see pause if say a job is scheduled to run on wifi, it starts, but then you switch to mobile. It's tricky though.

@enoch85
Copy link
Member

enoch85 commented Mar 14, 2017

@mario

Since the plan is to eliminate the separate Upload screen, Uploads/Syncs/Downloads/Whatever

So if I want to see if a file/folder was sucessfully uploaded I have to go to the servers activity app? I vote no! As a user I want to have some feedback directly in the app if the upload was successful or not, with the possibility to upload again if it failed hard. I don't entirely trust automated mechanisms. ;)

@AndyScherzinger
Copy link
Member

AndyScherzinger commented Mar 14, 2017

@enoch85 no, you to the client's activity view :)

...which will show you the uploads :D Where you can restart them, you don't loose any info present today in the upload view. You rather get an uploads+ on steroids view ;)

@enoch85
Copy link
Member

enoch85 commented Mar 15, 2017

@AndyScherzinger aah ok, well I rest my case. :)

@Bullnados
Copy link
Author

Bullnados commented Mar 23, 2017

@mario
@AndyScherzinger
Is that useful from an user standpoint to have just client's activity view? (Cue: usability)
If u can't guarantee that every file is 100% uploaded users will have a look into the app. So they have to click min. two extra times if NC app is on Homescreen(1.App-->2.client's activity view). Normally I would say the most users have NC in drawer menu or 2nd,3rd screen still more clicks and swypes.
That is not rly user-friendly.(cue: human-centered design)

Perhaps u should have a look at DIN EN ISO9241
Here is an extract of a thesis of mine

DIN EN ISO9241-12 sieben charakteristische Eigenschaften [15]:

  • Klarheit:
    • Schnelle Informationsvermittlung
  • Unterscheidbarkeit:
    • Genaue Differenzierung der Information
  • Kompaktheit:
    • Nur notwendige Information für die Aufgabe
  • Konsistenz:
    • Innerhalb der Anwendung gleichbleibende Bedienung
  • Erkennbarkeit:
    • Notwendige Information zieht Aufmerksamkeit des Benutzers auf sich
  • Lesbarkeit:
    • Informationen müssen leicht lesbar sein
  • Verständlichkeit:
    • Bedeutung der Information muss eindeutig, erkennbar, interpretierbar und leicht verständlich sein

@AndyScherzinger
Copy link
Member

Hi @Bullnados I get all the infos you posted but what is your suggestion here? Also as in "in contrast to today's solution" which is basically the same as the extension discussed above. The thing discussed here is just the app screen part. If I remember correctly we also have notifications as in Android notifications (which would/could then navigate you to the activities screen showing you the failed uploads, where I am uncertain if we do/don't do this today already) . Also we plan to work on hardening the upload system (like adding auto retries) so this should be an edge case in the future which doesn't change the fact that this might happen.

@Bullnados
Copy link
Author

Bullnados commented Mar 24, 2017

Hardening is always good and much more important than this cosmetic things. Perhaps I understood mario wrong, but he is against notifaction entirely? So the app is running in background and doesn't show any informations until the user is pressing the app button and switch to the "activity stream folder"?
I'm in #nextcloud-mobile now, perhaps I can explain it in german :)

This link i posted in IRC here are also some nice infos and more stuff to read
https://oshwiki.eu/wiki/Human_machine_interface

@tobiasKaminsky
Copy link
Member

I vote also to keep the notifications as I as an user wants to see if and what an app is doing.
There were already ideas to enhance this

  • see the outstanding uploads, eg. 2 out of 10, file abc.txt
  • pause an upload
  • and some other I do not remember

Having all of this in the activity view is also a good idea because you have than a upload/download history and can see what happened, because notifications always expire soon.
So I vote for having both, and extend both 👍

@mario
Copy link
Contributor

mario commented Apr 4, 2017

I would only show notifications when absolutely necessary.

Having an updating/persistent notification like we have it now requires a foreground service, and as-is it is a hack and a loophole that might be closed by Google soon enough anyway.

The goal is to completely eliminate the usage of our current upload/download service and move everything to jobs.

@Bullnados
Copy link
Author

Bullnados commented Apr 18, 2017

Mmh one importent point is. Android is not iOS. iOS are inured to have no control of their system. Androids are different especially the nextcloud users. They love to have full controll of the system and apps and knowing what the apps do.

👍 for more infos
😕 some simple infos
👎 for no infos

@joshtrichards joshtrichards added feature: activity and notification Server activity and notifications feature: transfers [WIP] Any uploading / downloading matters that aren't auto upload or sync specific labels Oct 9, 2023
@joshtrichards joshtrichards changed the title [Feature Request] Use notifcation bar to show upload status Use notifcation bar to show upload status Oct 16, 2023
@joshtrichards joshtrichards added hotspot: uploads tab and transfer notifications The uploads tab and upload + download related notifications and removed feature: activity and notification Server activity and notifications labels Dec 27, 2023
@joshtrichards joshtrichards added the needs review Potentially out of date or duplicate and/or fixed or mostly fixed so needs a status update label Oct 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement feature: transfers [WIP] Any uploading / downloading matters that aren't auto upload or sync specific hotspot: uploads tab and transfer notifications The uploads tab and upload + download related notifications needs review Potentially out of date or duplicate and/or fixed or mostly fixed so needs a status update
Projects
None yet
Development

No branches or pull requests

6 participants