Skip to content
This repository has been archived by the owner on Mar 19, 2024. It is now read-only.

[New Arch] Show + search + create private shares #254

Merged
merged 6 commits into from
Jul 5, 2019

Conversation

davigonz
Copy link
Contributor

@davigonz davigonz commented Jun 26, 2019

Implements #2555
Needs owncloud/android#2584

@davigonz davigonz changed the title [New Arch] Show + search + create private shares [WIP] [New Arch] Show + search + create private shares Jul 2, 2019
@@ -281,6 +281,9 @@ class ShareXMLParser {
} else if (name.equals(NODE_SHARE_WITH_DISPLAY_NAME, ignoreCase = true)) {
remoteShare.sharedWithDisplayName = readNode(parser, NODE_SHARE_WITH_DISPLAY_NAME)

} else if (name.equals(NODE_SHARE_WITH_ADDITIONAL_INFO)) {

Choose a reason for hiding this comment

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

Can you use here when instead of if, elseif...?

Copy link

@jcmontesmartos jcmontesmartos left a comment

Choose a reason for hiding this comment

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

LGTM

@davigonz davigonz merged commit c17c60a into new_arch/shares Jul 5, 2019
@delete-merged-branch delete-merged-branch bot deleted the new_arch/private_shares_get_create branch July 5, 2019 07:55
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.

3 participants