Skip to content

Alicia & Lak#13

Open
easternwashingaden wants to merge 100 commits intoAda-C13:masterfrom
easternwashingaden:master
Open

Alicia & Lak#13
easternwashingaden wants to merge 100 commits intoAda-C13:masterfrom
easternwashingaden:master

Conversation

@easternwashingaden
Copy link

Video Store Consumer

Congratulations! You're submitting your assignment!

Comprehension Questions

Question Answer
Paste a link to the PR for your API Ada-C13/video-store-consumer-api#9
For each person in your pair, describe something you learned from your partner during this project Alicia: Learned about good teamwork, we both worked really hard and kept pace with each other. Lak: Alicia is a bootstrap expert. Also, learned better way to user ternary.
What was one area of React you gained more clarity on during this assignment? Hook, callback functions, states, learned more about other useful gems/tools/libraries, API calls from Rails and external API.
Describe how you solved the problem of having lists of movies that look mostly the same but have different content We used ternary to display the different contents. If it is on the search page, it would have "Add" buttons. If it is on the library, it would have "Select" button.
Describe how you handled rentals with React We used state hook to store the states of selected movie and customer. The moment when we clicked on the "Checkout" button, the event handler would trigger the callback function, then the callback function got executed and created a new rental.
Describe a DOM event your application handled When we clicked on the Search button, the event listener triggered the callback function, the function got executed, and triggered rendering.
Did you use any functional stateless components? What for? Yes, We use functional stateless for all of our components.
Did you use any container components? What for? We were not sure what the container component means. We looked at a few resources, it referred to the classical component? If that's the case, we didn't use any of that.
Do you have any recommendations on how we could improve this project for the next cohort? More guidance on the deployment. It was difficult to write a front end in React after working on Rails for so long. Doing it in this order made it much more challenging.

easternwashingaden and others added 30 commits June 22, 2020 16:19
Changed class components to functional components
easternwashingaden and others added 30 commits June 24, 2020 21:47
Change notifications using react-notifications-component
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