Skip to content

[392] Add Notification Filters#458

Merged
Abd-Standard merged 3 commits into
Core-Foundry:mainfrom
susanyusuf:fix/392-add-notification-filters
Jul 26, 2026
Merged

[392] Add Notification Filters#458
Abd-Standard merged 3 commits into
Core-Foundry:mainfrom
susanyusuf:fix/392-add-notification-filters

Conversation

@susanyusuf

Copy link
Copy Markdown
Contributor

Summary

  • Add notification filters by type (discord/email/webhook/sms), delivery status, and date range on the Notification Search page
  • Wire filters end-to-end through searchNotifications and the listener search API
  • Keep Event Explorer read/unread filters unchanged

Test plan

  • Filter Notification Search by type and confirm results update
  • Filter by delivery status (PENDING, COMPLETED, FAILED, etc.)
  • Filter by date from/to and verify inclusive bounds
  • Confirm dashboard tests and listener search-filter tests pass

Closes #392

@drips-wave

drips-wave Bot commented Jul 25, 2026

Copy link
Copy Markdown

@susanyusuf Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

…y#392)

Allow Notification Search to filter by channel type, delivery status, and date range, wired through the listener search API. Also repair corrupted dashboard files so the preview build succeeds.
@susanyusuf
susanyusuf force-pushed the fix/392-add-notification-filters branch from 8989401 to f0a7c9c Compare July 25, 2026 09:07
susanyusuf and others added 2 commits July 25, 2026 11:33
Fork PRs cannot access CLOUDFLARE_API_TOKEN. Keep the dashboard build as the real gate and only deploy when the PR head is in the same repo.
@Abd-Standard
Abd-Standard merged commit 30bf21a into Core-Foundry:main Jul 26, 2026
1 check failed
@grantfox-oss grantfox-oss Bot mentioned this pull request Jul 26, 2026
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.

Add Notification Filters

2 participants