V0.0.2 changely api update

[ro-savage]

This PR updates NavMorph to handle Changelly's new API and return our new standardised API.

• [x] Re-write in use APIs to use {data: {}} response
• [x] Re-write and semi organise server code using async await
• [x] Update name to NavMorph
• [x] Simplify code on server side
• [x] Add prettier
• [x] Fix named imports to use correct naming convension
• [x] Add Error Log and Regular Log
• [x] Change from giving responses in deep functions and instead will always give response on top level of route
• [x] Remove passwords and replace with UUID
• [x] Remove dead code and speed up frontend
• [x] Update to Angular 5
• [x] Update jquery and Materialize to fix security issue
• [x] Other things I've forgotten.

Testing

• You will need to npm install to get new modules
• Ensure Mongod and daemon are running
• run npm start in frontend
• run npm start in server

Functional Testing

• Check homepage loads
• Create estimate
• Confirm estimate and send confirm
• See successful order places

Unit tests

• No new unit tests added or reviewed.
• Current unit tests may be broken as we undergo large changes

Code review

• Usual checks

[ro-savage]

Ready for review.

This is a lot of code, and way too much for one PR. But it ended up being a merge of a bunch of branches.

Good luck. Main thing to check is functionality. Can you type an address, get an estimate, send it, and get back the details.

[matt-auckland]

Probably worth leaving them for now, gives quick access to our orders if we even want to pick one to test with.

On 7/03/2018 14:52, "Ro Savage" notifications@github.com wrote:

@ro-savage commented on this pull request.

In server/src/api.js https://github.com/NAVCoin/NavMorph/pull/198#discussion_r172720152:

@@ -0,0 +1,30 @@ +"use strict"; + +const express = require('express') + +const router = express.Router([]) +const TransactionCtrl = require('./lib/db/transaction.ctrl') +const ChangellyCtrl = require('./lib/changelly/changelly.ctrl') +const OrderCtrl = require('./lib/order/order.ctrl') +const EtaCtrl = require('./lib/order/eta.ctrl') +const OrderStatusCtrl = require('./lib/order/status.ctrl') + +// TODO: Matt -- What are theses? Do they actually doing anything? +router.get('/db/transaction/:id', TransactionCtrl.getTransaction) +router.get('/db/transaction', TransactionCtrl.getTransaction) +

So we should delete them? OR do you want to keep them for testing?

— You are receiving this because your review was requested. Reply to this email directly, view it on GitHub https://github.com/NAVCoin/NavMorph/pull/198#discussion_r172720152, or mute the thread https://github.com/notifications/unsubscribe-auth/AMFcMhMjpttjkgIr6MqSBUI652yPXxLrks5tbz1EgaJpZM4SeAd9 .

[matt-auckland]

Code looks good, will test tonight when I'm home or tomorrow morning first thing.