Skip to content

feat: Add support for Box Hubs endpoints#664

Merged
arjankowski merged 3 commits intomainfrom
SDK-5824
Apr 16, 2026
Merged

feat: Add support for Box Hubs endpoints#664
arjankowski merged 3 commits intomainfrom
SDK-5824

Conversation

@arjankowski
Copy link
Copy Markdown
Contributor

@arjankowski arjankowski commented Apr 15, 2026

Added support for Box hubs endpoints:

Added also support for pagination in box-command.js markerPagination, as here we have the first endpoint added so far from ts sdk which requires pagination, but it's still not implemented in the sdk.

Additionally, I change in some commands the alias from -t to -T as this was not working before because of the conflict with the --token flag

@arjankowski arjankowski requested a review from a team April 15, 2026 15:24
@coveralls
Copy link
Copy Markdown

coveralls commented Apr 15, 2026

Coverage Report for CI Build 24514913458

Coverage increased (+0.2%) to 85.079%

Details

  • Coverage increased (+0.2%) from the base build.
  • Patch coverage: 2 uncovered changes across 1 file (146 of 148 lines covered, 98.65%).
  • No coverage regressions found.

Uncovered Changes

File Changed Covered %
src/commands/hubs/update.js 30 28 93.33%

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 5871
Covered Lines: 5226
Line Coverage: 89.01%
Relevant Branches: 2084
Covered Branches: 1542
Branch Coverage: 73.99%
Branches in Coverage %: Yes
Coverage Strength: 696.64 hits per line

💛 - Coveralls

Pask423
Pask423 previously approved these changes Apr 15, 2026
Copy link
Copy Markdown
Collaborator

@Pask423 Pask423 left a comment

Choose a reason for hiding this comment

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

👍

Comment thread src/commands/tokens/exchange.js
Copy link
Copy Markdown
Member

@congminh1254 congminh1254 left a comment

Choose a reason for hiding this comment

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

One comment

Copy link
Copy Markdown
Collaborator

@Pask423 Pask423 left a comment

Choose a reason for hiding this comment

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

👍

@arjankowski arjankowski merged commit 681b312 into main Apr 16, 2026
17 checks passed
@arjankowski arjankowski deleted the SDK-5824 branch April 16, 2026 14:23
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