Skip to content

Conversation

@manuelbahamondez
Copy link
Contributor

@manuelbahamondez manuelbahamondez commented Jan 12, 2026

Summary

  • Add new Inbox icon to the icon library

Test plan

  • Run npm run storybook - Inbox icon appears in BasicIcons story
  • Verify the icon renders correctly in Storybook
Screenshot 2026-01-12 at 2 46 59 PM

@manuelbahamondez manuelbahamondez marked this pull request as ready for review January 12, 2026 17:47
@manuelbahamondez manuelbahamondez requested review from a team and bvu-front January 12, 2026 17:47
@bvu-front bvu-front requested review from a team and maxgfront January 12, 2026 21:27
Copy link

@bvu-front bvu-front left a comment

Choose a reason for hiding this comment

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

LGTM, but also tagging dev-plat since I think they own this.

Copy link
Contributor

@kyle-pierce kyle-pierce left a comment

Choose a reason for hiding this comment

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

Addition of the icon looks fine. For releasing this, it looks like since you prefixed the PR with feat: a release PR will be created, and then you... close, open, and merge it to cause it to be released?

Looking at a previous example:

Image

@manuelbahamondez manuelbahamondez merged commit 421876b into main Jan 13, 2026
5 checks passed
@manuelbahamondez manuelbahamondez deleted the PB-66791-add-inbox-icon branch January 13, 2026 16:03
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