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- Jansen & Grace #15

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

Conversation

gracemshea
Copy link

Rideshare-Rails

Congratulations! You're submitting your assignment! These comprehension questions should be answered by both partners together, not by a single teammate.

Comprehension Questions

Question Answer
Describe the types of entity relationships you set up in your project and why you set up the relationships that way Driver has many trips, Passenger has many trips, and Trip belongs to Drivers and Passengers. We set it up this way because one Passenger and one Driver can have many trips. Whereas one trips only has one of each.
Describe the role of model validations in your application We used this to ensure that fields were completed correctly when submitted.
How did your team break up the work to be done? We started with a huge Trello and then claimed pieces of code, but we synced regularly and often debugged together.
What features did you choose to prioritize in your project, and what features, if any, did you have to set aside to meet the deadline? We prioritized the minimum functionality described in the Readme, and then went onto more complicated features.
What was one thing that your team collectively gained more clarity on after completing this assignment? 1. Routes 2. When it is appropriate to program separately and when it's best to come together on a problem.
What is your Trello board URL? https://trello.com/b/mfMjUwCf/rails-rideshare
What is the Heroku URL of your deployed application? https://tail-wagon.herokuapp.com/
What are two discussion points that you and your pair discussed when giving/receiving feedback from each other that you would be willing to share? We did discuss how we we approached debugging a little bit differently. We also learned about a really awful way to edit/push/pull/merge code and to avoid that forevermore.

gracemshea and others added 30 commits April 15, 2019 15:52
…' action. Create outline for 'create' action tests
…'create' action inside controller; Add test verifying it works. All tests passing.
JansenMartin and others added 30 commits April 18, 2019 19:23
…s; Add edit/delete actions for drivers; Add tests for drivers controller
…o test. All destroy tests currently passing.
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