Skip to content
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

Update Alertify #376

Open
ryanmartinneutrino opened this issue Apr 2, 2018 · 0 comments
Open

Update Alertify #376

ryanmartinneutrino opened this issue Apr 2, 2018 · 0 comments

Comments

@ryanmartinneutrino
Copy link
Collaborator

Currently, we've downloaded alertify.js into public and loaded it in a script tag in main.html.
We should remove it from main.html, and instead install it using npm.
Then, we need to update all obsolete API calls to alertify everywhere, and make sure that it's imported correctly every time that it's used (also, will help linter find corresponding errors)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant