Tidied up files

[developess]

All the additions and changes here are explained in #29

Summary:

• Removed erroneous file data.sql
• Removed superfluous import of http module in handler.js
• Removed reference to file until it has been written in the codealong (dynamic.js in handler.js)
• Removed duplicate step (1), reducing 5 steps to 4, renamed files
• Amended mentor notes to be clearer and renamed steps

Unforseen changes:

• In the process I noticed some inconsistencies in the db_build.sql files across steps - I amended them to be consistent with mentor notes.
• Some minor changes as a result of linting all files.

Closes #29

[eliasmalik]

@developess thanks this looks good, but I think some of the changes might conflict with the changes @mattlub made in #28 (specifically hoisting step-1 into root, but there may be others).

Happy to be advised by you both once you've coordinated on which PR to merge first.

[eliasmalik]

This either needs to be an if/else or there needs to be an early return in the if statement, otherwise an error will cause the callback to execute twice. Let's double check the rest of the steps too to see if it crops up in any of them too.

[mattlub]

Hi @developess, have you had a chance to see the PR #28? It doesn't change much but does some similar things to yours, but moving the starting code to the root directory, so students start coding from the root directory, and only need to look inside the steps folder for reference.

[developess]

@mattlub I didn't see it initially! I checked the issues but there was nothing about removing random files and imports. Happy for yours to be merged first and I'll see if there's anything left that I changed.

[arrested-developer]

@developess are there any outstanding changes you think need to be made?