Skip to content

[stable15] Fix the system address book - #12816

Merged
rullzer merged 1 commit into
stable15from
backport/12813/stable15
Dec 5, 2018
Merged

[stable15] Fix the system address book#12816
rullzer merged 1 commit into
stable15from
backport/12813/stable15

Conversation

@backportbot-nextcloud

@backportbot-nextcloudbackportbot-nextcloudBot commented Dec 4, 2018

Copy link
Copy Markdown

backport of #12813

@rullzerBackportbot
Fix the system address book
e5d445f
c23a66c broke the system address book.
We now move the ACL rules for this special case up and all is good in
the world again.
Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
@backportbot-nextcloudbackportbot-nextcloudBot mentioned this pull request Dec 4, 2018
Merged
MorrisJobke
MorrisJobke approved these changes Dec 4, 2018
View reviewed changes

@MorrisJobkeMorrisJobke left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code makes sense 👍

@MorrisJobkeMorrisJobke added the 3. to review Waiting for reviews label Dec 4, 2018
@MorrisJobkeMorrisJobke added this to the Nextcloud 15 milestone Dec 4, 2018
ChristophWurst
ChristophWurst approved these changes Dec 5, 2018
View reviewed changes

@ChristophWurstChristophWurst left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense!

@ChristophWurstChristophWurst added bug 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Dec 5, 2018
@rullzer
rullzer merged commit 13da727 into stable15Dec 5, 2018
@rullzer
rullzer deleted the backport/12813/stable15 branch December 5, 2018 07:42
@MorrisJobkeMorrisJobke mentioned this pull request Dec 5, 2018
Merged
12 tasks
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@MorrisJobkeMorrisJobkeMorrisJobke approved these changes
@ChristophWurstChristophWurstChristophWurst approved these changes
@georgehrkegeorgehrkeAwaiting requested review from georgehrke
@rullzerrullzerAwaiting requested review from rullzer

Assignees

No one assigned

Labels

4. to releaseReady to be released and/or waiting for tests to finishbug

Projects

None yet

Milestone

Nextcloud 15

Development

Successfully merging this pull request may close these issues.

3 participants

@MorrisJobke@ChristophWurst@rullzer