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

Cast share id to string in public auth #25402

Merged
merged 1 commit into from
Jul 11, 2016
Merged

Cast share id to string in public auth #25402

merged 1 commit into from
Jul 11, 2016

Conversation

PVince81
Copy link
Contributor

@PVince81 PVince81 commented Jul 7, 2016

Additional fix for #23066

@butonic @DeepDiver1975 @owncloud/sharing

Backports needed to:

  • stable9.1
  • stable9

@PVince81 PVince81 added this to the 9.2 milestone Jul 7, 2016
@mention-bot
Copy link

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

@PVince81
Copy link
Contributor Author

review please @DeepDiver1975 @butonic @georgehrke @VicDeo

@DeepDiver1975
Copy link
Member

👍

@georgehrke
Copy link
Contributor

lgtm 👍

@DeepDiver1975 DeepDiver1975 merged commit 2f22e31 into master Jul 11, 2016
@DeepDiver1975 DeepDiver1975 deleted the stringid-fix branch July 11, 2016 08:27
@DeepDiver1975 DeepDiver1975 self-assigned this Jul 11, 2016
@DeepDiver1975
Copy link
Member

Will take care about the backports

DeepDiver1975 pushed a commit that referenced this pull request Jul 11, 2016
@PVince81
Copy link
Contributor Author

stable9.1: #25432
stable9: #25431

@lock
Copy link

lock bot commented Aug 5, 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 5, 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