Skip to content

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

Merged
mergify[bot] merged 1 commit into
masterfrom
install-uploads-manager-shared-feature
May 19, 2026
Merged

chore(uploads-manager): install @box/uploads-manager shared feature#4563
mergify[bot] merged 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

Summary by CodeRabbit

  • Chores
    • Added uploads-manager (v1.14.0) to the project's development and peer dependency lists to align tooling and runtime expectations. No other dependency or configuration changes were made. This is an internal dependency alignment update and does not introduce new user-facing features or alter existing behavior.

Review Change Stack

@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

Note

Reviews paused

It looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the reviews.auto_review.auto_pause_after_reviewed_commits setting.

Use the following commands to manage reviews:

  • @coderabbitai resume to resume automatic reviews.
  • @coderabbitai review to trigger a single review.

Use the checkboxes below for quick actions:

  • ▶️ Resume reviews
  • 🔍 Trigger review

Walkthrough

Adds @box/uploads-manager (^1.14.0) to both devDependencies and peerDependencies in package.json.

Changes

Dependency Management

Layer / File(s) Summary
Add @box/uploads-manager to dev & peer deps
package.json
@box/uploads-manager (^1.14.0) added to devDependencies and peerDependencies.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

  • tjiang-box
  • jpan-box
  • jfox-box

Poem

🐇 I hopped through JSON, neat and spry,
added a manager with a tiny sigh,
two clean lines, no drama or clamor,
dev and peer both got their new hammer,
rabbit smiles — ready for the merger.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main change: adding @box/uploads-manager dependency to the project in a chore commit.
Description check ✅ Passed The description includes a clear summary, rationale for the change, detailed test plan with status indicators, and follows the repository's template structure with practical merge guidance.
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 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

socket-security Bot commented May 18, 2026

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

dlasecki-box
dlasecki-box previously approved these changes May 18, 2026
tjiang-box
tjiang-box previously approved these changes May 18, 2026
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented May 18, 2026

CLA assistant check
All committers have signed the CLA.

@dealwith dealwith force-pushed the install-uploads-manager-shared-feature branch from 21b90eb to b059f67 Compare May 19, 2026 09:19
@dealwith dealwith dismissed stale reviews from tjiang-box and dlasecki-box via 594d4e1 May 19, 2026 09:22
@dealwith dealwith force-pushed the install-uploads-manager-shared-feature branch from b059f67 to 594d4e1 Compare May 19, 2026 09:22
@dealwith dealwith force-pushed the install-uploads-manager-shared-feature branch from 594d4e1 to 50f2c79 Compare May 19, 2026 09:58
dlasecki-box
dlasecki-box previously approved these changes May 19, 2026
@dealwith dealwith force-pushed the install-uploads-manager-shared-feature branch 2 times, most recently from 78479dd to 1251893 Compare May 19, 2026 12:54
olehrybak
olehrybak previously approved these changes May 19, 2026
@dealwith dealwith force-pushed the install-uploads-manager-shared-feature branch from 1251893 to 422c5ec Compare May 19, 2026 13:04
@dealwith dealwith self-assigned this May 19, 2026
@dealwith
Copy link
Copy Markdown
Collaborator Author

Hey, @tjiang-box could you re-approve please

@olehrybak olehrybak requested review from tjiang-box and removed request for tjiang-box May 19, 2026 14:01
@mergify mergify Bot added the queued label May 19, 2026
@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented May 19, 2026

Merge Queue Status

  • Entered queue2026-05-19 15:56 UTC · Rule: Automatic strict merge
  • Checks skipped · PR is already up-to-date
  • Merged2026-05-19 15:57 UTC · at 422c5ec0672762ac970ef89127150a73d8a5bcc7 · squash

This pull request spent 46 seconds in the queue, including 9 seconds running CI.

Required conditions to merge

@mergify mergify Bot merged commit fa9ccea into master May 19, 2026
13 checks passed
@mergify mergify Bot deleted the install-uploads-manager-shared-feature branch May 19, 2026 15:57
@mergify mergify Bot removed the queued label May 19, 2026
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.

6 participants