-
Notifications
You must be signed in to change notification settings - Fork 182
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
enable private links #4599
enable private links #4599
Conversation
Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes. |
4f26a40
to
29b5c97
Compare
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just to be clear: this will show the private link
copy button in the right sidebar for each and every file in the web ui. Do we need some config.json option to prevent the web ui from showing the button? We need the capability for e.g. the desktop client. But the web ui behaves according to the capability as well. cc @pmaier1 @tbsbdr
@@ -0,0 +1,5 @@ | |||
Change: Enable privatelinks by default |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is this a breaking change? IMO only an enhancement
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why is change
always breaking?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Because the classification in calens maps very well to semver like that:
Security
andBugfix
forPatch
version incrementEnhancement
forMinor
version incrementChange
forMajor
version increment
Has been used in all web related repos like that since at least 1,5 years now. If not being used like that I don't understand the difference between Change
and Enhancement
to be honest. 😅
Same opinion. But since web respects the capabilities we'd need to add another config option in web to hide the private link copy button (which is ok). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, that matches the oC10 capability. LGTM 👍
will not be merged. We do not enable the feature user-facing. |
@michaelstingl does the desktop client need the private link capability? Or will it only check the presence of a private link prop in propfind responses? As of now, ocis has the private-link prop in propfind responses but announces the private link capability as disabled. For web this is fine. For other clients as well? |
No. Desktop and mobile clients show the UI for the private links depending on the private-link prop. |
Description
Enable private links capability.
Related Issue
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: