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 - Niv & Kirsten #22

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

Conversation

kanderson38
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 One trip has one passenger and one driver, one passenger has many trips, and one driver has many trips. This accurately reflected the relationships between the different entities.
Describe the role of model validations in your application The role of model validations is to make sure that the user needs to input data that the record requires to exist.
How did your team break up the work to be done? We communicated which tasks needed to be done, and divided them up in a way that seemed fair -- each partner did some code writing and some test writing.
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 meeting the functional requirements as laid out in the assignment. Styling was a lower priority -- we did some, but definitely could have done a lot more than we did.
What was one thing that your team collectively gained more clarity on after completing this assignment? We gained clarity on what it means to write a rails application and how the model, view, and controller interact with one another, especially when adding a feature. Kirsten also gained clarity on not accidentally opening a pull request on the AdaGold repo.
What is your Trello board URL? https://trello.com/b/kyfR4WSq/rideshare
What is the Heroku URL of your deployed application? https://niv-kirsten-rideshare.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? The driver/navigator method of working wasn't as prevalent in this project -- we would each work on a separate piece of the project simultaneously, which seemed to work well. Maybe we didn't use Trello as much as we should have, but it was easier to just talk to each other about what needed to be done. Overall, everything went very smoothly, and we enjoyed working with each other!

kanderson38 and others added 30 commits April 15, 2019 14:53
kanderson38 and others added 30 commits April 16, 2019 16:30


Merging trip#show and setting active status for driver
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