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

OCS Share API link shares now always have an url #16535

Merged
merged 1 commit into from
Jul 1, 2015
Merged

OCS Share API link shares now always have an url #16535

merged 1 commit into from
Jul 1, 2015

Conversation

rullzer
Copy link
Contributor

@rullzer rullzer commented May 22, 2015

Implementation of #16488

To make sure clients have the correct and latest link add an url field
to all OCS Share API return statements on public link shares.

  • Added unit tests

CC: @LukasReschke @schiesbn @PVince81 @MorrisJobke @DeepDiver1975

To make sure clients have the correct and latest link add an url field
to all OCS Share API return statements on public link shares.

* Added unit tests
@scrutinizer-notifier
Copy link

A new inspection was created.

@PVince81
Copy link
Contributor

Cool 👍

@ghost
Copy link

ghost commented May 22, 2015

Refer to this link for build results (access rights to CI server needed):
https://ci.owncloud.org//job/pull-request-analyser-ng-simple/12511/
🚀 Test PASSed.🚀
chuck

@MorrisJobke
Copy link
Contributor

Nice and works 👍

@schiessle
Copy link
Contributor

Makes sense 👍

@MorrisJobke
Copy link
Contributor

stable8.1 is created -> merge

MorrisJobke added a commit that referenced this pull request Jul 1, 2015
OCS Share API link shares now always have an url
@MorrisJobke MorrisJobke merged commit 8f05d4b into owncloud:master Jul 1, 2015
@rullzer rullzer deleted the ocs_share_api_url branch July 18, 2015 20:02
@lock lock bot locked as resolved and limited conversation to collaborators Aug 11, 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.

5 participants