Skip to content

Conversation

@cynthiacd
Copy link

Rails Zoo

Congratulations! You're submitting your assignment!

Comprehension Questions

Question Answer
Describe in your own words the Rails request/response cycle for viewing an animals detail page The user clicks on the animal name/link from the index page. This sends a request to the rails server. The rails server sends the request to the animals controller. The controller identifies that the show method needs to be called, for the animal id that came with the request. The animals controller asks the animal model to get the info from the db. Controller then asks the views to render a html show page for that animal. The views sends the html back to the controller which bundles it up as the response and sends it the user/browser.
Describe one area of Rails that you gained more clarity on with this assignment Felt a lot more confident with the routes and paths. I knew what to name them and how to access them when writing a link in views.
Is there a specific piece of code you'd like more feedback on? How to make the controller/view code more DRY. I made a helper method for my controller and put it in the private section - don't know if this is a good place for it
What is one aspect of this project that surprised you? When I generated the first migration I ended up created columns string and integer (somehow trying to declare my data types for name and age became their own columns). I don't know how I did this, but in the end felt this was a constructive mistake to make because I got to learn how to delete columns

@PilgrimMemoirs
Copy link

Rails Zoo

What We're Looking For

Feature Feedback
Appropriate Git usage with no extraneous files checked in Well Done
Answered comprehension questions Well Done - nice work creating a method for calling Animal.find
Naming conventions: controller is plural, model is singular Well Done
Overall

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