Skip to content

Replace native browser alerts with toast notifications #181

Merged
imanchalsingh merged 5 commits into
imanchalsingh:mainfrom
haniigupta:fix/auth-toast-notifications
Jun 26, 2026
Merged

Replace native browser alerts with toast notifications #181
imanchalsingh merged 5 commits into
imanchalsingh:mainfrom
haniigupta:fix/auth-toast-notifications

Conversation

@haniigupta

Copy link
Copy Markdown
Contributor

Description

This PR replaces the native browser alert() popups shown after successful login and registration with toast notifications using react-hot-toast.

Closes #168

Changes Made

  • Added react-hot-toast and configured the Toaster component.
  • Replaced the success alert on the Login page with a toast notification.
  • Replaced the success alert on the Register page with a toast notification.
  • Kept the existing authentication flow and redirect behavior unchanged.

Testing

  • Verified that a toast notification is displayed after a successful login.
  • Verified that a toast notification is displayed after a successful registration.

@netlify

netlify Bot commented Jun 25, 2026

Copy link
Copy Markdown

Deploy Preview for file-sharingsystem failed. Why did it fail? →

Name Link
🔨 Latest commit 85e8619
🔍 Latest deploy log https://app.netlify.com/projects/file-sharingsystem/deploys/6a3e1cf270da240008f7c46a

@vercel

vercel Bot commented Jun 25, 2026

Copy link
Copy Markdown

@haniigupta is attempting to deploy a commit to the imanchalsingh Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions Bot added the SSoC26 Official label of org label Jun 25, 2026
@github-actions github-actions Bot added bug Something isn't working frontend labels Jun 25, 2026
@imanchalsingh imanchalsingh added the Hard Point - 40 label Jun 26, 2026
@imanchalsingh imanchalsingh merged commit 5a8578a into imanchalsingh:main Jun 26, 2026
1 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working frontend Hard Point - 40 SSoC26 Official label of org

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Replace native browser alerts with modern toast notifications

2 participants