Skip to content

chore(uploads-manager): install @box/uploads-manager shared feature#4563

Open
dealwith wants to merge 1 commit into
masterfrom
install-uploads-manager-shared-feature
Open

chore(uploads-manager): install @box/uploads-manager shared feature#4563
dealwith wants to merge 1 commit into
masterfrom
install-uploads-manager-shared-feature

Conversation

@dealwith
Copy link
Copy Markdown
Collaborator

@dealwith dealwith commented May 18, 2026

Summary

  • Add @box/uploads-manager@^1.14.0 to dependencies and peerDependencies
  • Validates the shared feature builds and consumes correctly in Box UI Elements
  • No integration yet — wiring into ContentUploader will land in a follow-up PR behind a feature flag

Test plan

  • yarn install resolves @box/uploads-manager@1.14.0
  • yarn tsc --noEmit passes — types resolve (UploadsManager, UploadsManagerProps, UploadItem,
    UploadItemStatus, UploadsStatus)
  • Pre-push build (yarn build:es, yarn build:i18n) succeeds
  • CI green

@dealwith dealwith requested a review from a team as a code owner May 18, 2026 15:04
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 18, 2026

Walkthrough

This PR adds @box/uploads-manager (version ^1.14.0) to both devDependencies and peerDependencies in package.json, and reorders existing @box/content-field and @box/copy-input entries in both dependency sections.

Changes

Dependency Management

Layer / File(s) Summary
Add uploads-manager and reorder dependencies
package.json
@box/uploads-manager (^1.14.0) is added to devDependencies and peerDependencies. @box/content-field and @box/copy-input entries are reordered in both sections.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • box/box-ui-elements#4558: Both PRs modify package.json dependencies, including reordering of @box/copy-input in devDependencies and peerDependencies.

Suggested labels

ready-to-merge

Suggested reviewers

  • jpan-box
  • tjiang-box
  • tjuanitas

Poem

🐰 A new uploads friend joins the crew,
Dependencies ordered, fresh and true,
Neat and tidy, organized right,
Box-ui-elements shines so bright! ✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The pull request description includes helpful infrastructure guidance but lacks substantive technical details about the actual changes. Add a clear summary of what is being changed (e.g., 'Adding @box/uploads-manager@^1.14.0 to devDependencies and peerDependencies'), why it's needed, and the impact of these changes to align with best practices for pull request descriptions.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The pull request title clearly and specifically describes the main change: installing the @box/uploads-manager shared feature dependency.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch install-uploads-manager-shared-feature

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@socket-security
Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addednpm/​@​box/​uploads-manager@​1.14.0771001009650

View full report

@CLAassistant
Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


Hleb Kryshyn seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

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.

5 participants