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

FIRE - Pauline and Tram #9

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

Conversation

trambui09
Copy link

@trambui09 trambui09 commented Jan 21, 2021

Video Store Consumer

Congratulations! You're submitting your assignment!

Comprehension Questions

Question Answer
Paste a link to the PR for your API Ada-C14/video-store-consumer-api#5
For each person in your pair, describe something you learned from your partner during this project Tram - I learned that Pauline is great at manipulating the rails API to be able to add the video into the library. Pauline - Pauline learned alerts from Tram. The pop-up would show up on the screen when a user checks out and when a video is added to the library.
What was one area of React you gained more clarity on during this assignment? useEffect in watching for the updated video list. We needed to have a transition state that would capture the added video without infinitely rerendering the video list.
Describe how you solved the problem of having lists of movies that look mostly the same but have different content We had a movie(video) component and had those components render in our library list component.
Describe how you handled rentals with React With rentals, we had two states in our App that captures the current selected video and customer. When both of the states are selected (not empty) the checkout button appears and upon being clicked on, it would perform a post request to the rental controller action in our rails API.
Describe a DOM event your application handled For our search functionality, we had a form text box where the user could type in the video title they wanted to search for. Upon clicking search, the event is triggered to search the API and pull up matching results.
Did you use any functional stateless components? What for? For our Home component, there were no states involved. We just rendered the hero banner. Our library component is also stateless because the video list is updated and kept at the App level.
Did you use any container components? What for? Our App component holds the navbar and all the routes, it also holds the states of the current customer and current video selected along with the video list.
Do you have any recommendations on how we could improve this project for the next cohort? We feel like that project was scoped out the right amount in terms of possible workload and also liked the fact that it had optionals. Maybe for next cohort, Ada could include some CSS styling similar to the last projects.

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