Unfulfilled learning outcome

[eliasmalik]

In the README:

Running queries in Node

...

• Using parameterised queries (to prevent SQL injection)

I've looked through the walkthrough steps and there doesn't seem to be anywhere where this learning outcome is fulfilled. Step 3 Part 7 is the only place where the parameterised version of the Client.query command is written, but it's just an aside to the mentor.

I'd suggest either adding an INSERT query at the end of the code-along, or substituting the existing SELECT query for one.

[VirtualDOMinic]

Have requested the reviewers of the relevant pull request to merge and close

[mattlub]

@eliasCodes there is an old PR (#26 ) open to add an insert query, but maybe it would be better to just remove the learning outcome for now?

INSERT is covered in the workshop which follows this one: https://github.com/foundersandcoders/pg-workshop