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

Move post_removeFromGroup to shareManager #23973

Merged
merged 1 commit into from
Apr 19, 2016

Conversation

rullzer
Copy link
Contributor

@rullzer rullzer commented Apr 13, 2016

For: #22209

The last sharing hook to be moved over.

  • Added unit tests
  • Removed old tests that relied on old behaviour
  • Removed old hooks.php

CC: @schiesbn @nickvergessen @MorrisJobke @PVince81

The last sharing hook to be moved over.

* Added unit tests
* Removed old tests that relied on old behaviour
* Removed old hooks.php
@rullzer rullzer added this to the 9.1-current milestone Apr 13, 2016
@rullzer rullzer mentioned this pull request Apr 13, 2016
19 tasks
@nickvergessen
Copy link
Contributor

works 👍

@MorrisJobke
Copy link
Contributor

Tested and works 👍

@DeepDiver1975 DeepDiver1975 merged commit 1773dcb into master Apr 19, 2016
@DeepDiver1975 DeepDiver1975 deleted the share_move_post_delete_from_group_hook branch April 19, 2016 05:00
@lock
Copy link

lock bot commented Aug 6, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

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

Successfully merging this pull request may close these issues.

4 participants