Skip to content

fix : replaced alert call with toast notification in Dashboard.jsx - #1966

Open
tmdeveloper007 wants to merge 1 commit into
aryandas2911:mainfrom
tmdeveloper007:#1958
Open

fix : replaced alert call with toast notification in Dashboard.jsx#1966
tmdeveloper007 wants to merge 1 commit into
aryandas2911:mainfrom
tmdeveloper007:#1958

Conversation

@tmdeveloper007

Copy link
Copy Markdown

Summary of What Has Been Done

Replaced the browser alert() call in the routine duplication error handler with a toast notification, following the existing inline Toast component pattern.

Changes Made

  • Added useRef and useCallback imports
  • Added inline Toast component for non-blocking notifications
  • Added toast state and showToast helper with 3-second auto-dismiss
  • Replaced alert("Failed to duplicate routine") with showToast(..., "error")
  • Added Toast component to the JSX return

Impact it Made

  • Non-blocking UX during routine duplication failures
  • Consistent with toast notification pattern across the app

Closes #1958

Note: please assign this PR to the tmdeveloper007 account.

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.

fix : replace alert call with toast notification in Dashboard.jsx

1 participant