-
Notifications
You must be signed in to change notification settings - Fork 143
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
changing alert to toast message #195
Conversation
@MaximilianoCabreraP kindly resolve conflicts. |
Signed-off-by: Maximiliano Cabrera <[email protected]>
ok, i fix this =) |
hi, could you approve this PR? :D |
e@MaximilianoCabreraP kindly add |
@MaximilianoCabreraP what is this?? |
@MaximilianoCabreraP we do not want this? your PR is not ready i am sending to draft ask me when it will be ready. |
Hi, sorry, it was ready 2 weeks ago. Some new approved PRs broke mine. |
Sorry, I couldn't fix this since this PR, too many changes and conflicts. |
Description
I add toast.css file to style it.
I change all the alerts to the new "showToast" function that receives a message and a type (danger, warning and success)
At the bottom, I add the new features with manage the toasted message
Fixes #194
Type of change
Please delete options that are not relevant.
[ x ] New feature (non-breaking change)
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so that they can be reproduced.
Checklist: