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

Stacks - Jackie Rails Zoo #36

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

Conversation

jackiewatanabe
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 when you click on an animal or type in the url for a specific animal, then the router directs the request to the controller... which implements the show controller action... the show action then finds the specific animal based on the params id and then the view has the information of what should be displayed back to the browser
Describe one area of Rails that you gained more clarity on with this assignment How there are specific methods for migration files
Is there a specific piece of code you'd like more feedback on? whether I implemented adding images correctly
What is one aspect of this project that surprised you?

@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 Not quite
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