Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Griego PR for Trek #36

Open
wants to merge 12 commits into
base: master
Choose a base branch
from
Open

Griego PR for Trek #36

wants to merge 12 commits into from

Conversation

Trishthedish
Copy link

@Trishthedish Trishthedish commented Nov 29, 2016

Minimum Requirements

  • Wave 1
    Click button or link to show all trips ✅
  • Wave 2
    Can click on a trip, from the list of trips, to see more information about that trip ✅
    Should be able to see id, name, destination, continent, about, category, weeks and cost✅
  • Wave 3
    While viewing a single trip, you can reserve a spot ✅
    Use a form to submit your name to reserve a spot on the trip you are viewing ✅

Lessons Learned

  1. .serialize() on forms is looking for name="name" and is not looking for id="name". Next time I will use name ="whatever".
  2. (I should have realized sooner) that I was placing my ids name= (see above) on my html <section> tags. Lesson learned, placing naming conventions in the proper place. In this case on my <input> tags!
  3. Learned to better utilize my dev tools: command + shift + j > network > click on the file you wish to inspect > preview > so much to see! Found this article extremely helpful: https://developers.google.com/web/tools/chrome-devtools/network-performance/resource-loading for debugging

@PilgrimMemoirs @CheezItMan

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants