Hannah Cameron - Trek - Octos

[hannahlcameron]

TREK

Congratulations! You're submitting your assignment!

Comprehension Questions

Question	Answer
What does it mean for code to be asynchronous?	It means that not all the code is run at the same time - changes to the DOM occur only when certain events are triggered.
Describe a piece of your code that executes asynchronously. How did this affect the way you structured it?	There is the need to push through a reservation of a trip, but unless a trip is clicked, there is no trip to reserve. Within the document.ready, I had to ensure that the event handler for the submitting was outside the event handler that displayed specific trips. If the reservation event handler had been nested, clicking on the button wouldn't have worked to actually reserve a trip.
What kind of errors might the API give you? How did you choose to handle them?	The API could give errors if loading in the list of trips went awry, if loading in a specific trip didn't work (say a non-existing trip was requested) or if a reservation was submitted with incorrect information. I chose to handle these by displaying messages to the user that displayed a relevant message and then displayed the error messages from the API.
Do you have any recommendations on how we could improve this project for the next cohort?	nope.

[kariabancroft]

TREK

What We're Looking For

Feature	Feedback
Core Requirements
Git hygiene	Good
Comprehension questions	Yes
Functionality
Click a button to list trips	Yes
Click a trip to see trip details	Yes
Fill out a form to reserve a spot	Yes
Errors are reported to the user	Yes
Styling	Yes
Under the Hood
Trip data is retrieved using from the API	Yes
JavaScript is well-organized and easy to read	Yes
HTML is semantic	Yes, nice use of section and articles
Overall	You were able to utilize the API calls to complete the major requirements of this assignment. You are still missing a lot of semi-colons so it would be a good exercise to go back through this code and add them wherever they are missing.

[kariabancroft]

Watch out for little expressions like these where you're missing semi-colons

[kariabancroft]

Semi-colons on all of your userMessage function calls!

[kariabancroft]

A comment about what this code is doing would be good readability and reusability