Skip to content

Revert private links details param 10.0#1285

Merged
settermjd merged 2 commits into
10.0from
revert-privateLinksDetailsParam-10.0
May 29, 2019
Merged

Revert private links details param 10.0#1285
settermjd merged 2 commits into
10.0from
revert-privateLinksDetailsParam-10.0

Conversation

@phil-davis

Copy link
Copy Markdown
Contributor

The privateLinksDetailsParam was only added in 10.2
Revert parts of the changes that were backported to 10.0 in PR #1280

|`versioning`

|Navigating directly to a file's version, comments, and sharing pane
|`privatelLinks` and `privateLinksDetailsParam`

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I took these lines out completely, because I am not sure anyway what "Navigating directly" has to do with privateLinks - I will look into it...

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

privateLinks capability was added in core PR owncloud/core#29021
It refers to the capability that core DAV can return a private link to a file.
So I do not see what this has to do with "Navigating directly"

@settermjd

Copy link
Copy Markdown
Contributor

Thanks for creating the PR, @phil-davis.

@settermjd settermjd merged commit e5fbddd into 10.0 May 29, 2019
@delete-merged-branch delete-merged-branch Bot deleted the revert-privateLinksDetailsParam-10.0 branch May 29, 2019 08:50
@phil-davis

Copy link
Copy Markdown
Contributor Author

See PR #1286 for this revert in 10.1

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.

2 participants