Karinna Iniguez - Trek - Octos

[karinnainiguez]

TREK

Congratulations! You're submitting your assignment!

Comprehension Questions

Question	Answer
What does it mean for code to be asynchronous?	The code that we write is only applicable on certain actions, which means we do not load all of the functionality and code when the page loads.
Describe a piece of your code that executes asynchronously. How did this affect the way you structured it?	I have a button on my page that appears when the page loads. But when the button is clicked, it loads the list of trips. In order to do that, I had to add an event handler on the button when it's clicked.
What kind of errors might the API give you? How did you choose to handle them?	Creating a reservation can return an error because the appropriate fields were not filled in. I chose to show that message to the user in red text and clear the form.
Do you have any recommendations on how we could improve this project for the next cohort?	It's pretty great, but it feels more like a 3 day project than an entire week. Maybe shorten the time spent on this project.

[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	Good - nice grid practice
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
Overall	Overall you did a good job exploring the major learning goals of this assignment. Your code style and usage of JS syntax looks good.

[kariabancroft]

Good

[kariabancroft]

Desired style is to include this callback directly below the get function call - rather than adding this additional whitespace

[kariabancroft]

I like that you created the separate form function here. Might also be good to put the trip details HTML in a separate function as well.

[kariabancroft]

Interesting use of serialize!