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

Don't provide favorite activity settings #31081

Merged

Conversation

nickvergessen
Copy link
Member

Since mails and notifications are only available for actions of other users
it does not make sense to allow changing this.
It also prevents the common misunderstanding with
"file was changed inside a favorited folder"

Since mails and notifications are only available for actions of other users
it does not make sense to allow changing this.
It also prevents the common misunderstanding with
"file was changed inside a favorited folder"

Signed-off-by: Joas Schilling <coding@schilljs.com>
@artonge
Copy link
Contributor

artonge commented Feb 9, 2022

If users have enabled it, will they now be able to disable it?

@nickvergessen
Copy link
Member Author

If users have enabled it, will they now be able to disable it?

Since the setting does not have any effect at all, it does not matter?

@artonge
Copy link
Contributor

artonge commented Feb 9, 2022

For me, this setting was sending a notification when another user had access to one of my favourite folders and added or deleted a file.

If this is not the case, then why not remove it altogether?

@nickvergessen
Copy link
Member Author

This is not what the setting is about, this setting would kick in when someone else favorites or unfavorites a file for you. But they can not do that, as everyone can only do this for themselves.

Mails/push can not filter for favorites of others at the moment.

@artonge
Copy link
Contributor

artonge commented Feb 9, 2022

Thanks for the explanation. So we keep it in the hope that we implement that one day?

@nickvergessen
Copy link
Member Author

So we keep it in the hope that we implement that one day?

No, that is handled in FileFavoriteChanged

We never the less need to keep FavoriteAction so the messages are rendered in the stream

@nickvergessen
Copy link
Member Author

/backport to stable23

@nickvergessen
Copy link
Member Author

/backport to stable22

@nickvergessen
Copy link
Member Author

/backport to stable21

@eaCodet
Copy link

eaCodet commented Feb 18, 2022

For me, this setting was sending a notification when another user had access to one of my favourite folders and added or deleted a file.

This is exactly what I am looking for. The settings and the text suggest this behaviour. So if it worked at some point exactly this way why not make it work again?

@eaCodet
Copy link

eaCodet commented Dec 7, 2022

Still looking: notification for as favourites tagged directories. The settings and the text suggest this to work. Is there a way to make this work by now?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants