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

fix: stop using SharedLinkAPIConnection in getSharedItem() #1234

Merged
merged 6 commits into from
Feb 23, 2024

Conversation

lukaszsocha2
Copy link
Contributor

No description provided.

@coveralls
Copy link

coveralls commented Feb 21, 2024

Pull Request Test Coverage Report for Build #3626

Details

  • -3 of 11 (72.73%) changed or added relevant lines in 3 files are covered.
  • 9 unchanged lines in 2 files lost coverage.
  • Overall coverage increased (+0.5%) to 71.977%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/main/java/com/box/sdk/BoxAPIRequest.java 0 3 0.0%
Files with Coverage Reduction New Missed Lines %
src/main/java/com/box/sdk/BoxAPIRequest.java 1 81.48%
src/main/java/com/box/sdk/SharedLinkAPIConnection.java 8 0.0%
Totals Coverage Status
Change from base Build #3612: 0.5%
Covered Lines: 7500
Relevant Lines: 10420

💛 - Coveralls

Copy link
Contributor

@antusus antusus left a comment

Choose a reason for hiding this comment

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

Change the example to dive out the deprecated usage.

@@ -31,6 +31,7 @@ BoxItem.Info itemInfo = BoxItem.getSharedItem(api, sharedLink, password);
If you already know the type and ID of the item you want to access, you can
manually construct a [`SharedLinkAPIConnection`][shared-link-api] and make
API calls on the item directly.
Note: `SharedLinkAPIConnection` is now deprecated. Please use `BoxItem.getSharedItem()` method instead.

```java
String sharedLink = "https://app.box.com/s/abcdefghijklmnopqrstuvwxyz123456";
Copy link
Contributor

Choose a reason for hiding this comment

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

If we mark it as deprecated why not provide example with new usage? And show it in the documentation instead of deprecated one.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's a few lines above

Copy link
Contributor Author

Choose a reason for hiding this comment

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

do you think we should duplicate it here ?

Copy link
Contributor

@antusus antusus left a comment

Choose a reason for hiding this comment

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

👍

@lukaszsocha2 lukaszsocha2 merged commit 9f9af8e into main Feb 23, 2024
11 of 12 checks passed
@lukaszsocha2 lukaszsocha2 deleted the fix-getting-shared-items branch February 23, 2024 11:46
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.

3 participants