Skip to content

fix for sharing#4997

Merged
MorrisJobke merged 1 commit into
masterfrom
federated-share-fix
May 20, 2017
Merged

fix for sharing#4997
MorrisJobke merged 1 commit into
masterfrom
federated-share-fix

Conversation

@enoch85

@enoch85 enoch85 commented May 20, 2017

Copy link
Copy Markdown
Member

Signed-off-by: Daniel Hansson daniel@techandme.se

Fixes #3455

Signed-off-by: Daniel Hansson <daniel@techandme.se>
@mention-bot

Copy link
Copy Markdown

@enoch85, thanks for your PR! By analyzing the history of the files in this pull request, we identified @schiessle, @nickvergessen and @icewind1991 to be potential reviewers.

@enoch85

enoch85 commented May 20, 2017

Copy link
Copy Markdown
Member Author

Can someone backport to 12 and 11 as well please? Didn't figure out how to push to stable12, git said the branch was protected.

cc @karlitschek for approval

@nickvergessen

nickvergessen commented May 20, 2017

Copy link
Copy Markdown
Member

@enoch85 you need to create a branch based on stable12 and then send a PR against stable12

@enoch85

enoch85 commented May 20, 2017

Copy link
Copy Markdown
Member Author

@nickvergessen How do I fix the errors? Timeout...

@nickvergessen

Copy link
Copy Markdown
Member

You don't, its fine

@MorrisJobke MorrisJobke 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.

Tested and works 👍

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