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

Chantal's sanctuary #40

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

Chantal's sanctuary #40

wants to merge 16 commits into from

Conversation

ghost
Copy link

@ghost ghost 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 The request is made (to see an animal's details), it then goes to our routes to check and see which one matches. It then looks to the controller for the associated method that we need to access. The controller runs the method, and looks at the model (which contains our data), comes back with that info and sends it to the view/desktop so that we can see what it is we requested.
Describe one area of Rails that you gained more clarity on with this assignment The rails app diagram!
Is there a specific piece of code you'd like more feedback on? I'm curious to see how I should have handled the image size for each animal. The code is in my show.html file, lines 5-6, I display the right image for each animal, but couldn't figure out a good way to change the alt and the size without using an if statement.
What is one aspect of this project that surprised you? How fast I was able to move through it compared to the task list. I felt more confident and comfortable after some practice I guess. :)

@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 - glad you got some additional practice! I think for the image size, you could apply a class to that image within the image_tag helper, and set the sizes in a CSS file instead.
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.

1 participant