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

Merge notification, activities and uploads #1271

Open
mario opened this issue Aug 5, 2017 · 8 comments
Open

Merge notification, activities and uploads #1271

mario opened this issue Aug 5, 2017 · 8 comments
Labels
approved enhancement feature: activity and notification Server activity and notifications hotspot: uploads tab and transfer notifications The uploads tab and upload + download related notifications

Comments

@mario
Copy link
Contributor

mario commented Aug 5, 2017

No description provided.

@mario mario added this to the Nextcloud App 1.6.0 milestone Aug 5, 2017
@tobiasKaminsky
Copy link
Member

Merging Notifications and activities 👍 as they are both provided by the server.
Upload is only device specific and should stay separate:
If this is all sorted by date the failed/pending uploads will vanish somehow on the bottom of the list.
And if they stay on top of the list, then we have it just like the current way.

But maybe we just need a detailed input of @nextcloud/designers or from you if you thought already deeper about this.

@tobiasKaminsky tobiasKaminsky added enhancement needs info Waiting for info from user(s). Issues with this label will auto-stale. labels Aug 7, 2017
@mario
Copy link
Contributor Author

mario commented Aug 7, 2017

@tobiasKaminsky this was actually jointly designed by me and @jancborchardt, and there is a pen&paper wireframe in one of the issues (I'll find it for you later). I believe you also commented on it. :)

Also, stuff that needs actions have their own group so nothing really disappears.

@mario
Copy link
Contributor Author

mario commented Aug 7, 2017

(P.S. Many many months ago)

@jancborchardt
Copy link
Member

Reposting the spec with mockups from #557 (comment):

Here’s the mockup of the activity stream merged with notifications as @mario and I talked about:
img_20170124_113357

From the writeup in the sidebar issue: #556 (comment)

Activity: Merged Activity & notifications etc, much like the unified timeline in the sidebar of files nextcloud/server#658. Most of these entries will at some point be interactive, like accepting a share, reverting to a version, or commenting etc. Actionable items like failed uploads or shares to be accepted will be shown up top. They will be separated by intermittent headers:

  • Failed uploads is first cause it’s local and most important cause it can lead to data loss. Will have a »Retry« and »Clear« in the header with text+icons. Then icons for each individual item.
  • Requires attention (working title) are the notifications and activities with interactivity which come from the server. For example incoming shares you need to accept. (Or in the future comments where you were mentioned with an action to reply.)
  • Activities / History where all the activites go in the end, and where also the things you act upon from Failed uploads and Requires attention go to → which is a big reason why this is a single view instead of separate ones. It’s the same things.

@jancborchardt
Copy link
Member

To also answer your remark @tobiasKaminsky:

Upload is only device specific and should stay separate:
If this is all sorted by date the failed/pending uploads will vanish somehow on the bottom of the list.
And if they stay on top of the list, then we have it just like the current way.

Upload is only device specific until the file is uploaded. As soon as it’s uploaded it will morph to being in the activities view, hence uploads should be in there from the beginning. As described in the spec, failed/pending uploads should of course be first and will thus never vanish on the bottom of the list.

The point is that we have all in one view, without the mental work needed to remember »was it a local upload?« or »was it in activities now?«. It’s all just a feed of actions.

@tobiasKaminsky
Copy link
Member

Very good and detailed explanation, thank you @jancborchardt

@AndyScherzinger AndyScherzinger modified the milestones: Nextcloud App 2.1.0, Nextcloud App 2.2.0 Nov 9, 2017
@tobiasKaminsky tobiasKaminsky removed this from the Nextcloud App 3.1.0 milestone Feb 22, 2018
@tobiasKaminsky
Copy link
Member

As dicussed and agreed on, we do not plan with milestones.

@tobiasKaminsky tobiasKaminsky added approved and removed needs info Waiting for info from user(s). Issues with this label will auto-stale. labels Feb 22, 2018
@ashpieboop
Copy link
Contributor

From #4788, here's my input:

  • Rename Activities to Activity (unify with desktop)
  • Also display downloads (e.g. resulting from simple downloads, syncs and conflict resolutions)

I agree with #1271 (comment)

@joshtrichards joshtrichards added the feature: activity and notification Server activity and notifications label Oct 10, 2023
@joshtrichards joshtrichards added the hotspot: uploads tab and transfer notifications The uploads tab and upload + download related notifications label Dec 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved enhancement feature: activity and notification Server activity and notifications hotspot: uploads tab and transfer notifications The uploads tab and upload + download related notifications
Projects
None yet
Development

No branches or pull requests

6 participants