server icon indicating copy to clipboard operation
server copied to clipboard

Add multiple files to favorites

Open MariusBluem opened this issue 7 years ago • 11 comments

See proposed mockup:

bildschirmfoto 2017-12-26 um 18 35 06
  • Nextcloud 13

MariusBluem avatar Dec 26 '17 17:12 MariusBluem

I'll take this one.

weeman1337 avatar Jul 02 '18 19:07 weeman1337

I think this isn't as trivial as I thought in the beginning. It's more than just adding the action:

If the user selects only non-favorites: Action "Add to favorites" ✓

If the user selects only favorites: Action: "Remove from favorites" ✓

If the user select favorites and non-favorites?

  • Should we display both actions "Add to favorites" and "Remove from favorites"? That means if there are already some favorites and the user selects "add" the files simply keep their favorite status.
  • Not display the actions.

weeman1337 avatar Jul 22 '18 09:07 weeman1337

Most likely nothing for 14 -> moving to 15.

MorrisJobke avatar Jul 24 '18 09:07 MorrisJobke

@weeman1337 You still can continue to think about this and work on this, but it will then be integrated into 15 first.

MorrisJobke avatar Jul 24 '18 09:07 MorrisJobke

Once this is in, we should also do this on client-side

tobiasKaminsky avatar Apr 01 '21 12:04 tobiasKaminsky

Once this is in, we should also do this on client-side

You can already do this. It only needs a client side implementation. For multiple files/folder moves we trigger just multiple MOVE requests from the frontend, which allows proper UI feedback as well. The favorites already can be tagged via the PROPPATCH method. The only problem here is that the activity is not yet implemented. See #26393

MorrisJobke avatar Apr 01 '21 15:04 MorrisJobke

So there will be no new endpoint to pass bunch of file so mark them as favorited?

tobiasKaminsky avatar Apr 28 '21 10:04 tobiasKaminsky

So there will be no new endpoint to pass bunch of file so mark them as favorited?

Correct - only multiple PROPPATCH requests.

MorrisJobke avatar May 18 '21 07:05 MorrisJobke

@weeman1337 Any update here?

szaimen avatar Jul 06 '21 15:07 szaimen

This issue has been automatically marked as stale because it has not had recent activity and seems to be missing some essential information. It will be closed if no further activity occurs. Thank you for your contributions.

ghost avatar Aug 05 '21 15:08 ghost

Still a valid enhancement, no idea why bot closed this.

tobiasKaminsky avatar Aug 20 '21 05:08 tobiasKaminsky

@skjnldsv is there a specific new API to bulk add files to favorite, or is it "just" doing it one by one?

tobiasKaminsky avatar Jul 17 '23 09:07 tobiasKaminsky

For now one-by-one.

If I start adding dedicated APIs, I will ping the client team, I'll keep that in mind, no worries :) https://github.com/nextcloud/server/blob/0411cc64aaaaa42ae23b4d5e216f16d481f20819/apps/files/src/actions/favoriteAction.ts#L99-L102

skjnldsv avatar Jul 27 '23 15:07 skjnldsv