close
Skip to content

patch for transfer content to another user - #632

Merged
carycheng merged 3 commits into
masterfrom
transfer_folder_patch
Aug 9, 2018
Merged

patch for transfer content to another user#632
carycheng merged 3 commits into
masterfrom
transfer_folder_patch

Conversation

@carycheng

Copy link
Copy Markdown

No description provided.

@carycheng
carycheng requested a review from mattwiller August 8, 2018 22:23
@boxcla

boxcla commented Aug 8, 2018

Copy link
Copy Markdown

Hi @carycheng, thanks for the pull request. Before we can merge it, we need you to sign our Contributor License Agreement. You can do so electronically here: http://opensource.box.com/cla

Once you have signed, just add a comment to this pull request saying, "CLA signed". Thanks!

Comment thread doc/users.md Outdated
```

[move-folder-to-user]: http://opensource.box.com/box-java-sdk/javadoc/com/box/sdk/BoxUser.html#moveFolderToUser-java.lang.String-
[transfer-folder-to-new-user]: http://opensource.box.com/box-java-sdk/javadoc/com/box/sdk/BoxUser.html#transferFolderToNewUser-java.lang.String-

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This URL should probably be http://opensource.box.com/box-java-sdk/javadoc/com/box/sdk/BoxUser.html#transferContent-java.lang.String-

Comment thread doc/users.md Outdated
BoxUser user = new BoxUser(api, "0");
BoxFolder.Info folderInfo = user.moveFolderToUser("1");
String destinationUserID = "12345";
BoxUser sourceUser = new BoxUser(api, "0");

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Maybe pull out a separate variable for sourceUserID to make this example more explicit

@coveralls

Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 1709

  • 11 of 11 (100.0%) changed or added relevant lines in 1 file are covered.
  • 1 unchanged line in 1 file lost coverage.
  • Overall coverage increased (+0.1%) to 58.236%

Files with Coverage Reduction New Missed Lines %
src/main/java/com/box/sdk/EventStream.java 1 87.5%
Totals Coverage Status
Change from base Build 1705: 0.1%
Covered Lines: 4497
Relevant Lines: 7722

💛 - Coveralls

@carycheng
carycheng merged commit 9273d77 into master Aug 9, 2018
@carycheng
carycheng deleted the transfer_folder_patch branch August 9, 2018 02:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants