-
Notifications
You must be signed in to change notification settings - Fork 54
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
WIP: add support for link share preview #475
Conversation
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.
To me the wording 'Preview' is very misleading, I would vote for 'View only' at least in the UI.
Other than that, LGTM
Kudos, SonarCloud Quality Gate passed! |
@mrow4a what should we do here in the Serverteam board? |
@NannaBarz decide whether we want to have this feature or not.. ref. https://github.com/owncloud/enterprise/issues/5277 |
@hodyroff this is not in the richdcuments 4.0.0 release ticket. Should it be added? |
@jnweiger this here was PoC. Also requires Core change. |
@mrow4a would it be a lot of effort to finalize this + owncloud/core#40289 based on the original approach? Would be very nice to have this in core 10.13 + richdocuments 4.0.0. |
No, good POC. But the use case for secure view/no download are very secret documents. Why would a user show those through public links, rather then accounts which can be audited and tracked? We continue the work when this comes up as a requirement again. |
@hodyroff initial Request is not actual anymore ? If so, I'd also avoid this feature or disable it by default. |
Yes, not valid anymore. |
WARNING: do not merge, yet this is in Proof of Concept state only