-
Notifications
You must be signed in to change notification settings - Fork 220
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
[UI] Upgrade packages and engines #625
Conversation
Cheers 🍻 @pietdaniel ! I appreciate the help, I'm a noob when it comes to JS and keeping the libs up to date. Def can use help you can provide modernizing the JS I created a PR against this one to update the node versions in the ITs and locally |
Head branch was pushed to by a user without write access
Codecov Report
@@ Coverage Diff @@
## master #625 +/- ##
=======================================
Coverage 83.43% 83.43%
=======================================
Files 16 16
Lines 1443 1443
=======================================
Hits 1204 1204
Misses 207 207
Partials 32 32 Continue to review full report at Codecov.
|
Should be final fix required to get Integration Tests to pass
@all-contributors please add @pietdaniel for code |
I've put up a pull request to add @pietdaniel! 🎉 |
This PR updates the following ui packages:
It also updates the following engines:
Some small changes were made to the
webpack.*.conf.js
files in order to resolve issues in building locally.Due to the
buefy
upgrade, thesyntax was no longer valid, being replaced with:
This PR updates all references to the modern style.
Finally, a lint error on the location of some attributes and the
beforeRouteLeave
function was resolved.This PR has been tested and built locally and deployed to a live environment.