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

Ports - Kim and Alex #19

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

Conversation

alexandria7
Copy link

@alexandria7 alexandria7 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 Ada-C11/VideoStoreConsumer-API#8
For each person in your pair, describe something you learned from your partner during this project Alex: Kim is really great at debugging and sticking with a problem until it's fixed! This is something I often struggle with so it was really inspiring and helpful to have a partner who consistently troubleshoots. She was also great at keeping coding fun (she often added neat lines of code that made our code more concise and our product generally better!)! // Kim: Alex did a great job of directing the flow and pace of the project, and keeping up with the Trello board (something I struggle with remembering to update), so it was really nice to have her as an example for that! She also did a great job of not putting the styling off until the end of the project (like I definitely tend to do...), and implementing small amounts throughout, which made styling decisions much easier near the end, and our site look nice and polished!!
What was one area of React you gained more clarity on during this assignment? Alex: I learned how to send information to a back-end (i.e our Rails API). // Kim: I learned about setTimeout and how to make messages disappear (bye messages!). // Both: We learned about React Router!
Describe how you solved the problem of having lists of movies that look mostly the same but have different content We had two different components for rendering our lists - SearchMovie for our list of searched movies and MovieList for our rental library movies. Both utilized the Movie component for displaying individual movies.
Describe how you handled rentals with React Within our React app we let the user select a customer and a movie. Once both were selected a "check out" button appeared that caused a post request to be sent to our rails API with information about the movie, customer, and due date. If it was successful we sent a success message.
Describe a DOM event your application handled When a user clicks "Add To Library" in the Movie Search page, a click event called "onSearchResultClick" gets called which then calls the callback function "addSearchToLibraryCallback" from App. In App, the "addSearchToLibrary" function gets called which sends a POST request to our API to add that particular movie to the movie database. This also sets the state of App to update our front-end movie library.
Did you use any functional stateless components? What for? Movie, Customer, CustomerList, and MovieList all are functional components. This was because they didn't need to have their own state and all of the information they held was passed from props.
Did you use any container components? What for? App was a container component - it pulled information from our API and set its state with that information. It then passed that information to the presentational components for rendering.
Do you have any recommendations on how we could improve this project for the next cohort? It felt pretty smooth to us! :D

Kimberly-Fasbender and others added 30 commits June 24, 2019 13:35
alexandria7 and others added 30 commits June 26, 2019 13:51
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