I like the way you have organised it - drop all tables at the top if they exist, then create new ones. It makes a bit more sense than ours.

You probably don't need the NOT EXISTS condition after it though:

DROP TABLE IF EXISTS people, lunch, bookings;

CREATE TABLE IF NOT EXISTS people (

Tiny, inconsequential thing. Ill raise a more important issue next time!

Could use CHAR(50); to limit the amount of text.

The way that you created your xml http request function, and then used the dom manipulation function as an argument seems super concise and modular!

You should make sure you end your response so the user doesn't get taken to your 404!

something like this at the end of postUserHandler would do the trick...

  response.writeHead(301, { location: '/' });
  response.end();

Remember to add instructions on setting up environment variables for your project in your readme.md. You should mention how they are added (file name) and what variables are needed.

You did it! Very impressed, because we didn't manage any 😅

The tests also pass, and I like the console logging of the test output to be super clear.

My only suggestion is that the error messages could be more descriptive, as "this is a test" is a little vague 🤓

Whether it will stop SQL injection I don't have a clue, but it would be a quick win. Throw something like this inline in the text input(s)

pattern="[a-zA-Z\s]{0,40}"

This regular expression will allow for for a-z, upper and lower case, as well as white space, to be entered. You can then add (or remove) an error class with javascript on the fly, listening to the 'input' event on the form fields. Use:

e.target.validity.valid

For more info on this

You probably know this, because the hard part is actually getting the lunch_id and user_id from the other tables!

This is the query that won't work at the moment:

So to make it work:
I'm guessing you have a pre-populated lunches table. You will need to:

• Add the user - This currently works so WELL DONE!
• Get the lunch id
• Get the user id
• use the lunch id, user id and paid variable to add them to the booking table!

BUT - these operations are asynchronous, so they will need to be chained together through nested callbacks! (I know, hell). I can try and help you with this this morning if you want!

Just a minor thing - I know that we're still in the middle of development so ignore this if it's just because of that, but there seems to be a lot of extraneous console logs and huge chunks of commented code, so evaluating which ones are still necessary might be useful!

(We totally did the same thing though 😅)

Even if we came up with it first 😜🤣😘 (KIDDING!)

You can use the same db_build.sql for testing and building your actual databases. In fact, its probably better to keep them as one so you're sure you don't have any differences between them.

If those files have small differences, then you're not testing your real database. You're testing a different database!!

All looks really organised and super easy to get up and running so THANKYOU!

On form submit in dom.js, you can run an XMLHttpRequest() to post information to your server, which will get entered into the database:

xhr.open("POST", "/urlToListenForInRouter", true);
xhr.send(yourInformationvariableHere);

You can then listen for a POST request and the url to listen for in the router.js:

const method = request.method;

if (method === "POST" && url === "/urlToListenForInRouter") {
    handleYourFormDataHandler() // - this looks like your postUserHandler() in handlers.js
}

It does still seem to work, because you get your dates, but I think it has something to do with the response that you put into your getLunch function. When console logging Array.isArray(response) returns false 3 times before returning true. Similarly, logging the typeof the response returns string 3 times before returning object (screenshots below). It looks like the code runs 4 times, failing 3 times before the input type is changed and it actually gets to load. Maybe you know all this, but if not I hope it helps to figure out whatever is happening!

console.log(typeof(response)):

console.log(Array.isArray(response)):

This would be a cool feature to see