Skip to content

feat: integrate Solvro Alerts banner#352

Merged
qamarq merged 6 commits intobetafrom
feat/solvro-alerts
May 5, 2026
Merged

feat: integrate Solvro Alerts banner#352
qamarq merged 6 commits intobetafrom
feat/solvro-alerts

Conversation

@Antoni-Czaplicki
Copy link
Copy Markdown
Member

@Antoni-Czaplicki Antoni-Czaplicki commented May 1, 2026

Summary

  • Adds drop-in <Alerts /> component (mounted once in src/app/layout.tsx) that fetches alerts from https://alerts.solvro.pl/api/v1/alerts/?app=<APP_CODE>, with a 60s TanStack Query cache.
  • Sanitises HTML content via isomorphic-dompurify against an inline-only allow-list (defence-in-depth on top of the server's nh3 pass).
  • Renders banners stacked at the top, color/icon by alert_type (info/warning/critical), with optional dismiss persisted to localStorage (solvro-alerts-dismissed).
  • Whole banner is clickable when link is set (external links open in a new tab).
  • App code defaults to planer; overridable via NEXT_PUBLIC_ALERTS_APP_CODE env var.
image image

@Antoni-Czaplicki Antoni-Czaplicki marked this pull request as draft May 1, 2026 11:42
@Antoni-Czaplicki Antoni-Czaplicki marked this pull request as ready for review May 2, 2026 08:49
@Antoni-Czaplicki Antoni-Czaplicki requested a review from qamarq May 2, 2026 08:49
@qamarq
Copy link
Copy Markdown
Member

qamarq commented May 4, 2026

@Antoni-Czaplicki z laski swojej na stronie glownej jakos nizej i ladniej to wkomponuj, na stronie planu jest spoko

@Antoni-Czaplicki
Copy link
Copy Markdown
Member Author

No zamysł jest taki żeby to było na górze jednak, na stronie plany trochę schowane bardziej żeby nie przeszkadzało już w trakcie używania

@qamarq
Copy link
Copy Markdown
Member

qamarq commented May 4, 2026

@Antoni-Czaplicki to dodaj go zamiast standardowego topwr jesli jest dostepny. z tym samym designem pls. bo ten alert jest calkowicie z dupy tam na gorze

@Antoni-Czaplicki
Copy link
Copy Markdown
Member Author

@qamarq ten styl badge topwr się jak dla mnie nie nadaje do tego - za bardzo się wtapia w design a alerty mają być używane jeśli mamy coś ważnego do przekazania i powinny zwracać na siebie uwagę, mogę go trochę upodobnić do pigułki topwr ale tak jak jest zrobione teraz to idealnie krótki alert się komponuje w przerwie między navbarem a cta

@qamarq
Copy link
Copy Markdown
Member

qamarq commented May 4, 2026

@Antoni-Czaplicki dobrze upodobnij go, ogranicz dlugosc etc

@Antoni-Czaplicki
Copy link
Copy Markdown
Member Author

done:
image

@qamarq
Copy link
Copy Markdown
Member

qamarq commented May 5, 2026

@Antoni-Czaplicki czy jesli sie zamknie baner to pigulka z topwr dalej jest w tej samej odleglosci od gory czy nagle jest tuz przy samej gorze? i tlo zrob blur background plus jakis kolorek np czarny polprzezroyczty dal ciemnego motywu i bialy dla jasnego

@Antoni-Czaplicki
Copy link
Copy Markdown
Member Author

@qamarq pigulka topwr jest nie ruszana

@Antoni-Czaplicki
Copy link
Copy Markdown
Member Author

blur nie będzie na razie zbyt widoczny bo nic nie ma pod spodem, ale dodałem na przyszłość, kolor tła jest lekkie opacity z koloru poziomu alertu

image 2026-05-05_10-17-32 image image

Copy link
Copy Markdown
Member

@qamarq qamarq left a comment

Choose a reason for hiding this comment

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

lgtm

@qamarq qamarq merged commit 12cff12 into beta May 5, 2026
3 checks passed
@qamarq qamarq deleted the feat/solvro-alerts branch May 5, 2026 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants