Skip to content

[PB-6491]:feat/usage advertisements#112

Merged
jaaaaavier merged 13 commits into
masterfrom
feat/usage-warning-banner
Jun 19, 2026
Merged

[PB-6491]:feat/usage advertisements#112
jaaaaavier merged 13 commits into
masterfrom
feat/usage-warning-banner

Conversation

@jaaaaavier

Copy link
Copy Markdown
Contributor

This PR includes the design of the new storage usage banner and everything that entails: new colours, a new story, new colours, exporting the component, updating the package version, test...

We have also updated the SidenavStorage component, which features some minor design updates and for which we have created its own story

jaaaaavier and others added 6 commits June 9, 2026 14:30
yarn publish prompts for version confirmation interactively, causing the CI job to hang and fail silently.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@jaaaaavier jaaaaavier self-assigned this Jun 18, 2026
@jaaaaavier jaaaaavier added the enhancement New feature or request label Jun 18, 2026
@jaaaaavier jaaaaavier marked this pull request as ready for review June 18, 2026 08:10
@jaaaaavier jaaaaavier requested a review from xabg2 June 18, 2026 08:10
Comment thread src/components/feedback/notifications/usageBanner/UsageWarningBanner.tsx Outdated
Comment thread src/components/feedback/notifications/usageBanner/UsageWarningBanner.tsx Outdated
Comment thread src/components/navigation/sidenav/SidenavStorage.tsx Outdated
Comment thread src/components/navigation/sidenav/SidenavStorage.tsx Outdated
Comment thread src/components/navigation/sidenav/SidenavStorage.tsx Outdated
@jaaaaavier jaaaaavier requested a review from xabg2 June 18, 2026 12:04
Comment on lines +56 to +60
<div className="flex flex-row items-center gap-2">
<div className="h-3 w-8 rounded-lg bg-gray-5 animate-pulse" />
<div className="h-3 w-2 rounded-lg bg-gray-5 animate-pulse" />
<div className="h-3 w-8 rounded-lg bg-gray-5 animate-pulse" />
</div>

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Extract this to a Skeleton component

Comment on lines +19 to +23
<div className="flex flex-row items-center gap-2">
<div className="h-3 w-8 rounded-lg bg-gray-5 animate-pulse" />
<div className="h-3 w-2 rounded-lg bg-gray-5 animate-pulse" />
<div className="h-3 w-8 rounded-lg bg-gray-5 animate-pulse" />
</div>

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Use a Skeleton component

@jaaaaavier jaaaaavier requested a review from xabg2 June 18, 2026 14:30
@jaaaaavier jaaaaavier merged commit 874e289 into master Jun 19, 2026
1 check passed
@jaaaaavier jaaaaavier deleted the feat/usage-warning-banner branch June 19, 2026 07:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants