feat(usage overview): Add breakdown into panel #104141
Conversation
github-actions
bot
added
the
Scope: Frontend
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
| }); | ||
|
|
||
| const formattedAdditionalReserved = null; | ||
| const formattedGifted = displayPriceWithCents({cents: reservedBudget.freeBudget}); |
There was a problem hiding this comment.
Bug: Gifted field displays zero values unnecessarily
The formattedGifted variable is always computed regardless of whether freeBudget or free is zero. Since displayPriceWithCents({cents: 0}) returns "$0.00" and formatReservedWithUnits(0, category) returns "0" (both truthy strings), the conditional {formattedGifted && ...} will render the "Gifted" field showing "$0.00" or "0" when there's actually no gifted amount. The tests explicitly set non-zero gifted values, suggesting this zero case wasn't intended to display. Compare this to formattedAdditionalReserved which correctly uses a conditional check before formatting.
Additional Locations (1)
There was a problem hiding this comment.
this was intentional however since this is not GA'd yet it's possible we will provide the same treatment as we do for additional reserved or vice versa later on
Split out from #103983
Pt 5 of https://linear.app/getsentry/issue/BIL-1817/update-usage-overview-table-with-new-designs
Introduces the new panel (behind a non-existent FF) + fixes file structure so components are nested
