Console.logs shouldn't really be in the master branch.
When there are a bunch of console.logs scattered through an app it usually means there aren't any (or enough) unit tests. I know it's hard to get into the mindset of TDD but it would have been perfect for this project, especially the logic file.
The issues with error handling would be easier to diagnose and fix if you were writing tests as you go.
Console.logs shouldn't really be in the master branch.
When there are a bunch of console.logs scattered through an app it usually means there aren't any (or enough) unit tests. I know it's hard to get into the mindset of TDD but it would have been perfect for this project, especially the logic file.
The issues with error handling would be easier to diagnose and fix if you were writing tests as you go.
tests tests tests tests