Skip to content

Conversation

@nickvergessen
Copy link
Member

@rullzer can you please take care of adding tests?

@mention-bot
Copy link

@nickvergessen, thanks for your PR! By analyzing the annotation information on this pull request, we identified @rullzer, @PVince81 and @DeepDiver1975 to be potential reviewers

@rullzer
Copy link
Member

rullzer commented Jul 28, 2016

O man that is nasty indeed.
I'll take care of the tests.

@rullzer rullzer self-assigned this Jul 28, 2016
@schiessle schiessle added 2. developing Work in progress and removed 3. to review Waiting for reviews labels Jul 29, 2016
@schiessle
Copy link
Member

setting to "developing" until tests are added

@rullzer rullzer force-pushed the h1-prevent-delete-update-on-group-shares branch from 4ef0a5c to a6d6b26 Compare August 2, 2016 09:01
@rullzer
Copy link
Member

rullzer commented Aug 2, 2016

And we have an intergration test!
🎉

@schiessle please review

@rullzer
Copy link
Member

rullzer commented Aug 2, 2016

👍 for @nickvergessen commit

@rullzer rullzer added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Aug 2, 2016
@rullzer rullzer removed their assignment Aug 2, 2016
@schiessle
Copy link
Member

looks good 👍

@nickvergessen
Copy link
Member Author

nickvergessen commented Aug 2, 2016

👍

@rullzer rullzer force-pushed the h1-prevent-delete-update-on-group-shares branch from a6d6b26 to 1088870 Compare August 2, 2016 09:12
@MorrisJobke MorrisJobke merged commit 17c3027 into master Aug 2, 2016
@MorrisJobke MorrisJobke deleted the h1-prevent-delete-update-on-group-shares branch August 2, 2016 12:17
GitHubUser4234 pushed a commit to GitHubUser4234/server that referenced this pull request Aug 30, 2016
…ate-on-group-shares

Do not allow to delete/update group shares as a group member
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants