Skip to content

Homework Complete#6

Open
fbonds wants to merge 5 commits intofrontend-application-development-uw20:masterfrom
fbonds:master
Open

Homework Complete#6
fbonds wants to merge 5 commits intofrontend-application-development-uw20:masterfrom
fbonds:master

Conversation

@fbonds
Copy link

@fbonds fbonds commented May 1, 2020

Week 3 HW Submission

Please fill out the information below in order to complete your assignment. Feel free to update this comment later if necessary.

  • Comfort rating on this assignment (1-5): 2
  • Completion rating on this assignment: complete-ish. I really struggled with State and, as have no design experience to speak of, spent inordinate amounts of time just on trying to get it to look right. I never could quite sort out how to get the cart to show w/ individual items or place it to the right of the properties. Tim helped me here, butI still struggled with getting that aspect to work. Finally, I opted instead to make a cart total appear/hide on the NavBar. I hope that suffices.

<div className="item">
{airbnbs.map((vacationObj, index) => {
vacationObj.key = index;
return <VacationRentals {...vacationObj} />

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit: if the component is for a single item, call it VacationRental

}

static propTypes = {
vacationObj: PropTypes.shape({

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Really nice work here!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants