Skip to content

Conversation

marisol-lopez
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 So the animals are listed on the index page, and when you click on a specific animal, it will re-route you to the animal's page by their ID. So when the user clicks on the animal's name, I have already told the program to animal_path by ID which is going to redirect to the show action and then the show.html.erb page
Describe one area of Rails that you gained more clarity on with this assignment Using Ruby in my html.erb files; I didn't initially understand code tags vs. value tags to complete Ruby actions
Is there a specific piece of code you'd like more feedback on? Two things, I was fiddling around in my rails console, and I was trying to change the ID of an animal I had created, but I was unable to do that. Second, more clarification on the button. I understand that it will automatically want to be a POST, so I have to be explicit and say method: :get, but what is the class: "button" doing. Because I am saying button_to and so why do I need to say the class: button?
What is one aspect of this project that surprised you? How the show method had to be after certain other methods and routes, because it affected my program.

@PilgrimMemoirs
Copy link

Rails Zoo

What We're Looking For

Feature Feedback
Appropriate Git usage with no extraneous files checked in Well Done
Answered comprehension questions Well Done - where in there does the model come in?
Naming conventions: controller is plural, model is singular Well Done
Overall
Here's a resource on how to change it, http://stackoverflow.com/questions/8301635/how-to-change-primary-id-of-a-record-in-rails
For the button, you do not need to put the class portion in there. It still works without. It is commonly included so that your button tag has the class of button, which some frameworks use to give a button a specific style.

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