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 groupmates also when renaming and moving files (by using Group folders app) #500

Closed

Conversation

ghost
Copy link

@ghost ghost commented Sep 23, 2020

Currently:
Groupmates (using Group folders app) are only notified when a file is created.

With this PR:
Notify groupmates also when renaming and moving.

Thanks in advance for a feedback.

@ghost ghost changed the title update FilesHooks class Update: Notify groupmates also when renaming and moving files (by using Group folders app) Sep 23, 2020
@ghost ghost changed the title Update: Notify groupmates also when renaming and moving files (by using Group folders app) Notify groupmates also when renaming and moving files (by using Group folders app) Sep 23, 2020
@ghost ghost force-pushed the update/group_users_notifications branch from a1677c3 to 843f830 Compare September 23, 2020 10:38
@ghost
Copy link
Author

ghost commented Sep 23, 2020

Hi @nickvergessen !
Pls when you have time, maybe you can also review this PR for me? Thanks a lot in advance.

lib/FilesHooks.php Outdated Show resolved Hide resolved
@ghost ghost force-pushed the update/group_users_notifications branch from 843f830 to 84f5180 Compare September 23, 2020 11:12
Signed-off-by: Chris Fritsche <chris.fritsche@cunio.de>

adjusts from PR review

adjust notifications by moving
@ghost
Copy link
Author

ghost commented Oct 2, 2020

Partly solves nextcloud/groupfolders#1018.

@fschrempf
Copy link
Contributor

@chrisfritsche @nickvergessen Would you mind to respin this and bring it over the finish line?

@nickvergessen
Copy link
Member

Fine by me. I'm time limited and won't be able to do it myself.

@fschrempf
Copy link
Contributor

Here is a follow-up PR: #693.

@fschrempf fschrempf closed this Jan 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants