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

Sockets Tatiana and Grace (Gratiana) #16

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

Conversation

tatsqui
Copy link

@tatsqui tatsqui commented Jun 28, 2019

Video Store Consumer

Congratulations! You're submitting your assignment!

Comprehension Questions

Question Answer
Paste a link to the PR for your API
For each person in your pair, describe something you learned from your partner during this project Tatiana learned from Grace how to create a Search component and integrate that with the external api. Grace got more comfortable following tracing errors through the JS console with Tatiana’s help.
What was one area of React you gained more clarity on during this assignment? Using Axios with React. Learning how to use the Router module as well.
Describe how you solved the problem of having lists of movies that look mostly the same but have different content Our movie component displays information specific to each movie, like title, overview, image and release date. When we map each movie into our movie list, its individual contents are rendered properly.
Describe how you handled rentals with React App.js is the parent component for Customer List and Rentals. With callback functions we added the Customer ID, and the Rental title to state in App, and once both of those pieces of information were gathered, the user can then make a rental reservation, which was handled with an eventHandler which makes a post request to our server.
Describe a DOM event your application handled When a user submit a movie title in the search box, the state of the MovieSearch component is updated via onFormSubmit. A GET request is sent to the database with the searchQuery of the movie title.
Did you use any functional stateless components? What for? Movie component and Customer component were both functional stateless components.
Did you use any container components? What for? Yes - Library and CustomerList components were containers of Movie and Customer respectively.
Do you have any recommendations on how we could improve this project for the next cohort? The movie_wrapper in the Rails API doesn’t play nicely with image URLs. The API expects a path without a base URL. Evelynn pointed this out, the fix was a conditional in the scaffolded method.

tatsqui and others added 30 commits June 24, 2019 15:35
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