feat: filter incidents with impact none out of banner by charislam ¡ Pull Request #43311 ¡ supabase/supabase
No actionable comments were generated in the recent review. đ
âšī¸ Recent review info
Configuration used: Central YAML (base), Repository UI (inherited), Organization UI (inherited)
Review profile: CHILL
Plan: Pro
Cache: Disabled due to Reviews > Disable Cache setting
Disabled knowledge base sources:
- Linear integration is disabled
You can enable these sources in your CodeRabbit configuration.
đ Files selected for processing (1)
apps/studio/components/layouts/AppLayout/useStatusPageBannerVisibility.ts
đ Walkthrough
Summary by CodeRabbit
- Bug Fixes
- Improved status page banner display logic to exclude non-impactful incidents, ensuring the banner only appears when there are incidents with actual impact.
Walkthrough
The useStatusPageBannerVisibility hook has been updated to filter incidents from the status page events. Specifically, incidents with an impact level of 'none' are now excluded from the set of incidents evaluated for banner visibility. The incidents variable is renamed internally, and the filtering logic is applied before determining whether active incidents exist. The public API of the hook remains unchanged.
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.
Comment @coderabbitai help to get the list of available commands and usage tips.