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

Notify changes for favorites files #12872

Closed
PVince81 opened this issue Dec 16, 2014 · 6 comments
Closed

Notify changes for favorites files #12872

PVince81 opened this issue Dec 16, 2014 · 6 comments

Comments

@PVince81
Copy link
Contributor

Add new activity option "File or folder that is a favorite has changed".

Not sure how this would work.
Does it mean non-favorited files should not have activity entries any more ?

@karlitschek @nickvergessen

@karlitschek
Copy link
Contributor

this would be an additional option. so for example a user could get updates of all files in the activity stream but emails only for the favorites.

@nickvergessen
Copy link
Contributor

Let's figure out the details tomorrow then

@nickvergessen
Copy link
Contributor

Okay, after having some thoughts and a discussion at lunch, we came to the conclusion,
that this is not easily possible.

The problem is, we need to get the shared file, then for each user check whether the file or a parent folder is marked as favorite. Since the paths may be different for each user, this means at least a few queries per user (check tag for file, [get parent folder, check tag for parent,]* ...), so e.g. in an installation with thousands of users, we run thousands of additional queries, just to find out, that we don't need to do anything.

So unless someone has a good idea how to better do this, I would abstain from this atm.

@RobinMcCorkell
Copy link
Member

If this is still desired, please open a new issue over at owncloud/activity.

@nickvergessen
Copy link
Contributor

No, actually activities are not part of the activity app. The activity app is just an empty framework without any context. Also there is already a PR open:
#15196

@nickvergessen
Copy link
Contributor

Implemented in 8.1

@lock lock bot locked as resolved and limited conversation to collaborators Aug 9, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants