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

sai #18

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

sai #18

wants to merge 5 commits into from

Conversation

ssamant
Copy link

@ssamant ssamant commented Apr 2, 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 user sends a request from the browser to view a particular page (based on id of animal). that request goes to to the router, which sends the path on to the controller with the controller action "show". the show action calls the method in the model that finds the animal info in the database and brings it back to the controller. the controller then sends that info out to the show page of the views -- which populates the show view with ERB html. at some point (in the controller??) the ERB html gets converted to regular html and that is what the user sees in the browser.
Describe one area of Rails that you gained more clarity on with this assignment better understanding of the paths in the routes and keeping track of them
Is there a specific piece of code you'd like more feedback on? no
What is one aspect of this project that surprised you? maybe not surprising but still learning/memorizing all the steps to build a rails project.

@ssamant ssamant closed this Apr 3, 2017
@ssamant ssamant reopened this Apr 3, 2017
@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
Naming conventions: controller is plural, model is singular Yes
Overall Nice job!

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