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

[Feature] Respect private link capability #3738

Merged
merged 6 commits into from
Aug 30, 2022

Conversation

abelgardep
Copy link
Contributor

@abelgardep abelgardep commented Aug 25, 2022

Related Issues

App: #3732

Library PR (if needed): owncloud/android-library#505

  • Added changelog files for the fixed issues in folder changelog/unreleased. More info here

QA

@abelgardep abelgardep force-pushed the feature/private_link_capability branch from a506382 to a745542 Compare August 26, 2022 12:22
@abelgardep abelgardep marked this pull request as ready for review August 26, 2022 12:25
@JuancaG05 JuancaG05 self-requested a review August 26, 2022 12:47
Copy link
Collaborator

@JuancaG05 JuancaG05 left a comment

Choose a reason for hiding this comment

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

Great work! LGTM 👍

@jesmrec
Copy link
Collaborator

jesmrec commented Aug 29, 2022

Three cases to check:

  • privateLink: false(oCIS) -> private link not available ✅
  • privateLink: true -> private link available ✅
  • No privateLink in capabilities -> private link available ✅

@jesmrec
Copy link
Collaborator

jesmrec commented Aug 29, 2022

(1) [FIXED]

Something weird. Is the platform (oCIS/oC10) involved in the logic?

in oCIS: "privateLinks": false -> no private link
in oC10: "privateLinks": false -> private link available.

Samsung A51 Android12
a745542e4

@jesmrec
Copy link
Collaborator

jesmrec commented Aug 29, 2022

(2) [WONT FIX HERE]

This case is also interesting to take in account:

  1. In server admin dashboard, open Admin -> Sharing
  2. Disable Allow apps to use the Share API

in that case, Share API is not available and users will not be allowed to shares with users or by link. But, the "Share" option is still available because the private link.

I mean, if:

  • Share API is disabled
  • privateLinks: false

the sharing option in top bar could be disabled as well.

This is an improvement, not a bad behaviour

Samsung A51 Android12
a745542e4

@abelgardep
Copy link
Contributor Author

@jesmrec
(1) Should be fixed now.
(2) It's not that simple to implement at the moment. I think we can create a new issue to do that

@jesmrec
Copy link
Collaborator

jesmrec commented Aug 29, 2022

@abelgardep after fixing (1), if privateLink capability is not retrieved, private links are not available.

@jesmrec
Copy link
Collaborator

jesmrec commented Aug 29, 2022

(1) fixed
(2) addressed to #3740

Feature is approved

@abelgardep abelgardep merged commit d625d04 into feature/open_in_web Aug 30, 2022
@abelgardep abelgardep deleted the feature/private_link_capability branch August 30, 2022 08:06
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.

[FEATURE REQUEST] [oCIS] Respect Capability "privateLinks": false
3 participants