Skip to content

Queues - Cara Comfort - rails-zoo #22

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 9 commits into
base: master
Choose a base branch
from

Conversation

cecomfort
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's name to get more information on that animal, a get request is sent to the uri animals/:id. The router then takes that request and passes it along to the show method in the animals controller. The animals controller retrieves the object corresponding to that animal from the model and then goes to the view to look for show.html. That html is then sent to the browser, which renders the html so you can view the information on the animal you selected.
Describe one area of Rails that you gained more clarity on with this assignment The difference between the new and create routes/methods.
Is there a specific piece of code you'd like more feedback on? Is it necessary to create a partial for the form if you want to allow different attributes for create and update? What's the DRYest way to go about this?
What is one aspect of this project that surprised you? I remembered more than I thought I would!

@PilgrimMemoirs
Copy link

Rails Zoo

What We're Looking For

Feature Feedback
Appropriate Git usage with no extraneous files checked in Well done - nice commit messages!
Answered comprehension questions Well Done
Naming conventions: controller is plural, model is singular Well Done
Overall
The partial should be used to include as much repeated code as possible, so you can include or exclude as much as you like to still allow any customization. So if there are only some fields of your form that you want to have the same, you can use your partial to only include those fields and then build the rest of the form in the view files.
Clever delete button

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