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

Jou-Jou's Rails Zoo #23

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

Jou-Jou's Rails Zoo #23

wants to merge 11 commits into from

Conversation

jjousun
Copy link

@jjousun jjousun commented Apr 3, 2017

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 First, the browser makes a HTTP GET request for the URL named, which is animals/:id. The router maps the URL to the correct controller and action to handle the request, which is Animal and show. The action receives the request, and asks the model to retrieve data from the database (table of animals). The model returns a list of data to the controller action (the specific animal user requested). The controller action passes the data on to the view. The view renders the page as HTML. The controller sends the HTML back to the browser. The page loads and the user sees it.
Describe one area of Rails that you gained more clarity on with this assignment I think I need more practice doing this over and over again. I still have some trouble with the named paths. Also, the fact that the order of the routes matters (must go from specific to general).
Is there a specific piece of code you'd like more feedback on? Not sure.
What is one aspect of this project that surprised you? It was kind of fun! I even tried to practice some more CSS.

@kariabancroft
Copy link

Rails Zoo

What We're Looking For

Feature Feedback
Appropriate Git usage with no extraneous files checked in Yes
Answered comprehension questions Yes - I'll quote you on this one... "It was kind of fun!" LOL
Naming conventions: controller is plural, model is singular Yes
Overall Nice work!

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