search

BYU-ODH / yvideo-client

Front-end code for Y-Video (React)
GNU General Public License v3.0
3 stars 3 forks source link

774 Change all alert message for sweetalert message #813

Closed jtabilot closed 1 year ago

jtabilot commented 1 year ago

Change all the alert native message for Sweetalert

jtabilot commented 1 year ago

I prefer avoiding new dependencies whenever possible. Can this be done without sweetalert2? For example, since we are already adopting react-bootstrap, does it include the functionality we need?

@reynoldsnlp I gree with you about avoid including new depencies, however sweetalert was already installed in the proyect. On other hand, as far as I know bootstrap dosen't have native modals like sweetalert, I mean you need to define the modal in your html code, instead sweetalert gives you all the code already implemented. Now answering your question about if this can be done witout sweetalert, yes, but it will implicate to write an html modal.