search

NextFaze / ionic-manup

Mandatory Update for Ionic
MIT License
52 stars 27 forks source link

Bugfix/28 alert metadata #29

Closed MichaelMarner closed 6 years ago

MichaelMarner commented 6 years ago

Tests to confirm #28 and then a fix.

Note the setTimeout in the test is a bit gross, but it's because ManUp is designed to not resolve the promise if an alert is required.

coveralls commented 6 years ago

Coverage Status

Coverage increased (+9.9%) to 78.07% when pulling 7742e04cda47e89690696cf970d82acc8d229554 on bugfix/28-alert-metadata into dbc83f30d31986173c3b23138fae66005f51b390 on master.