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

Branches - Paige & Caroline #21

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

Conversation

stupendousC
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-C12/video-store-consumer-api#14
For each person in your pair, describe something you learned from your partner during this project Caroline learned from Paige: incorporating lifecycle methods to solve page load issues. Paige learned from Caroline: using Helpers.js to carry in fcns that's needed across diff component files
What was one area of React you gained more clarity on during this assignment? How frontend and backend are related, and how they share info via API calls.
Describe how you solved the problem of having lists of movies that look mostly the same but have different content Not sure what you mean, but the movies still retain their external_id's from whence they came.
Describe how you handled rentals with React We made a new component, which contains the array of overdueRentals from the API call at first. Later we also added allRentals from a new API endpoint we made. When user clicks on the Rentals route, they'll see the default view of allRentals in a table, but they can toggle between seeing allRentals or overdueRentals.
Describe a DOM event your application handled In the Customers component showing customers list, if the user clicks on the "SELECT" button next to a specific customer, it will send that customer's info up to App.js. App.js then sets state for CurrCustomer with that person's info, which then displays on the page.
Did you use any functional stateless components? What for? We did at first, but because we wanted to use lifecycle methods, we had to change all our functional components to classical components
Did you use any container components? What for? Per css-tricks.com: Container Components are almost always the parents of Presentational Components. In a way, they serve as a intermediary between Presentational Components and the rest of the application. They’re also called “smart” components because they’re aware of the application as a whole. So App.js is a container component. Because it contains all the important state info, and passes the relevant props down to its child components.
Do you have any recommendations on how we could improve this project for the next cohort? Maybe a tutorial on how to deploy? We encountered problems because AWS didn't like the ruby language versions we used.

stupendousC and others added 30 commits December 16, 2019 12:52
successfully imported inventory[] and customers[] into App.js front end
destructured props in Customer.js
Movie & Library components
…s with checkboxes to allow us to add to rental library
external movie search API successful.  next up: print out those resul…
…ically, if none selected it will say so, else will give customer & movie id/name
minipaige02 and others added 30 commits December 18, 2019 12:31
…sibly gonna build refreshing state into componentWillMount() for Rentals/Customers/Movies comps
Customer details scaffolding
Completed show customer details and styling
Refactored customer component creation
DRY'd out BASE_URL into helper file
fixed isOverdue() bug, now compares dates with Date.parse() instead o…
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