Skip to content

Queues- Danielle Birbal - Rails-Zoo #41

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

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

Conversation

birbalds
Copy link

@birbalds birbalds 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 user clicks on the "Get to know me more" link that sends a request that is routed through rails and sent to the controller. The controller gathers the correct data on the specific animal from the Model and the correct viewing format from the Views then combines them and pushes out a response to the browser.
Describe one area of Rails that you gained more clarity on with this assignment This assignment helped me better understand image tags.
Is there a specific piece of code you'd like more feedback on? No
What is one aspect of this project that surprised you? When making the form for the 'Species' field I tried to implement a editable drop-down list. Apparently its very difficult to make an editable drop down list with just rails and heavily advised to use javascript to accomplish this.

@PilgrimMemoirs
Copy link

Rails Zoo

What We're Looking For

Feature Feedback
Appropriate Git usage with no extraneous files checked in Could use more commits
Answered comprehension questions Well Done
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