[Bug] Notification Service 'update' Toasts Use Incorrect Class Name #2997
Labels
Bug
A defect in the app
Dev Approved
The issue has been approved by a team member and can be worked on.
Dev Complete
Release Pending
Issue has been resolved and is waiting to be packaged as part of a release
Is there an existing issue for this?
Current Behavior
Update notification alerts use a
toastClass
of "warn" when "warning" was likely intended.ngToast/Bootstrap-css provide for several alert classes:
["success", 'info", "warning", "danger"]
, of which "warn" is not in this list. Additionally, "warn" does not appear anywhere else in the code base, excluding logger.warn and an (unused) Twitch warn event. As such, this was likely a mere misspelling. However, it's possible that this is in-fact not a bug, and is intended behavior.ngToast docs (unversioned)
Bootstrap 3.4 docs
Expected Behavior
Toast that is properly browned, but not burned.
Steps To Reproduce
Version
5.64-sources
What operating system are you using Firebot on?
Windows 10
Relevant log output
No response
Contact Details (Optional)
No response
The text was updated successfully, but these errors were encountered: