Skip to content

PR for Rolodex Project #33

Open
Trishthedish wants to merge 19 commits intoAda-C6:masterfrom
Trishthedish:master
Open

PR for Rolodex Project #33
Trishthedish wants to merge 19 commits intoAda-C6:masterfrom
Trishthedish:master

Conversation

@Trishthedish
Copy link

@Trishthedish Trishthedish commented Dec 6, 2016

@PilgrimMemoirs & @CheezItMan:

  • Still working on completing all requirements for wave 3 about 98.79% complete. Here is my current PR request as per requested via slack.

  • A) I thought the live code that I was a part of was extremely helpful.

  • B) I was sick and had to play some catch up. However, I found the following series of videos extremely helpful: backbone.js tutorial. Future students, might find these videos helpful to digest the weekend before you start backbone. Each video is < 8 mins long & always ends with a code summary of what you should have learned.

When wave 3 is complete your application should:

  • Allow the user to select a Contact by clicking upon its card in the contacts list.
  • Have a contact details modal. It should:
    - Be displayed when a contact card is selected.
    - Use the provided template for contact details.
  • Fill in the template with attributes from the selected Contact.
  • Be removed from the page when any other part of the page is clicked upon. ◻️ ❗️

…ection. This list should update when a new entry is added to collection.
…render to page. Earlier code now works. Missing class call on button-click
…and how and why its not rendering to page. appending the info.
…'re blank. Though when you inpsect them. They have proper data. Likely something wrong with how I'm calling something.
… what was happening in my pull request to merge this branch. Now its ready to merge\!
…uld improve how its displaying them. Validating correct input. And handling when someone attempts to save a blank contact
details are displaying on page.
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