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

Bianca & Angela |Pipes| VideoStoreConsumerBackbone #5

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

Conversation

biciclista22
Copy link

Video Store Consumer

Congratulations! You're submitting your assignment!

Comprehension Questions

Question Answer
For each person in your pair, describe something you learned from your partner during this project Angela: Deeper level of puzzling and practicing talking out loud through the thought process. Bianca: Before starting the project, checking in about expectations and needs for the pair project to be successful. I was also impressed by the desire to take on a challenge, such as CSS/styling, that was outside of Angela's comfort zone.
What was one area of Backbone you gained more clarity on during this assignment? We learned how important listeners are in the initialize of a model view. We tried to cause a view change in an event method without realizing there was no listener set to rerender the view.
Describe how you solved the problem of having lists of movies that look mostly the same but have different content We kept the movie view and the Backbone template, but when we added a movie to the library, we added an if/else statement in the template to handle the different situation of local rental library versus large API call to external API.
Describe how you handled rentals with Backbone? We did not attempt this optional.
Describe a DOM event your application handled We called a render on moviesLibraryView when the view library button is clicked.
Describe a custom event your application handled Our add event. It is linked to the add button. It saves the model to the rails API.
Do you have any recommendations on how we could improve this project for the next cohort? I appreciate that basic functionality was the baseline with more challenging optionals available. It was just the right amount of challenge given depleted mental reserves :)

@Hamled
Copy link

Hamled commented Jan 5, 2018

Video Store Consumer

What We're Looking For

Feature Feedback
Core Requirements
Git hygiene - pair contribution
Comprehension questions
General
Search functionality
Rental library listing functionality
Add to rental library functionality
Underscore Templates
Overall Look & feel
Optionals
Overall

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.

3 participants