Skip to content

feat(frontend): toast notifications + error boundary - #56

Merged
Pa004 merged 2 commits into
mainfrom
feat/notifications
Sep 1, 2026
Merged

feat(frontend): toast notifications + error boundary#56
Pa004 merged 2 commits into
mainfrom
feat/notifications

Conversation

@Pa004

@Pa004 Pa004 commented Sep 1, 2026

Copy link
Copy Markdown
Owner

Changes

  • ToastProvider with success/error/info toasts (4s auto-dismiss)
  • Shows toast on analysis complete (success with count or error)
  • ErrorBoundary wraps app with retry UI

- ToastProvider with success/error/info toasts (4s auto-dismiss)
- Shows toast on analysis complete (success with count or error)
- ErrorBoundary wraps app with retry UI
- Fixed main.tsx to include both providers
@coderabbitai

coderabbitai Bot commented Sep 1, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Team

Run ID: f8662249-e029-458c-9115-c7dc437c3493


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

# Conflicts:
#	frontend/src/App.tsx
#	frontend/src/main.tsx
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying sentinel with  Cloudflare Pages  Cloudflare Pages

Latest commit: 94f0c2a
Status:⚡️  Build in progress...

View logs

@Pa004
Pa004 merged commit b13e1a2 into main Sep 1, 2026
6 of 8 checks passed
@Pa004
Pa004 deleted the feat/notifications branch September 1, 2026 22:49
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.

1 participant