Skip to content

Ashton's Zoo #34

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 30 commits into
base: master
Choose a base branch
from
Open

Ashton's Zoo #34

wants to merge 30 commits into from

Conversation

@ashtn
Copy link

@ashtn ashtn 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 Sorry BRB
Describe one area of Rails that you gained more clarity on with this assignment Understanding where to look to resolve errors
Is there a specific piece of code you'd like more feedback on? no thank you
What is one aspect of this project that surprised you?

ashtn added 30 commits March 29, 2017 19:05
@ashtn
baseline met
6f2b390
@ashtn
Animals Controller and Animal Model generated, schema defined. Root r…
a414a09 
…oute set to index. Index action defined in Animals controller
@ashtn
Defined /animals route, Initiated index controller#action, displayed …
f515cc1 
…list of current animals in index view tempalte
@ashtn
created route to show one animal
4fa0077
@ashtn
defined show method in controller
7c46f33
@ashtn
created show view template
12fc8fb
@ashtn
initialized show controller#action
9725fce
@ashtn
Listed Animal name, species and age on show view tempalte. fixed show…
11468aa 
… error
@ashtn
added random animal image to show view template
6f6aa28
@ashtn
added link to each animal in the index to the associated animal show …
e4e8a14 
…page
@ashtn
added new and createroutes
1b0ece4
@ashtn
added link to create new animal on index view template
0922071
@ashtn
initialized new controller#action
153a8e0
@ashtn
added partial form view template
08565a0
@ashtn
fixed partial form errors
12af966
@ashtn
created new view template and rendered partial form to create new ani…
eca7b48 
…mal post
@ashtn
initialized create controller#action, defined strong params
a5b1770
@ashtn
fixed create controller#action bug
f32b445
@ashtn
initialized edit and update controller#actions
dec5a79
@ashtn
added edit and update routes
e615717
@ashtn
created edit view template, rendered partial form
336bf03
@ashtn
Fixed route error in update
a431693
@ashtn
adjusted formatting in edit view template
4d90a75
@ashtn
initialized destroy controller#action
106271d
@ashtn
created delete animal route
079dde7
@ashtn
fixed delete route bug
75451d3
@ashtn
fixed controller action destory spelling error
7c4c89d
@ashtn
added delete button to show view tempalte
c03f794
@ashtn
added edit button to show view template
42315cd
@ashtn
added home button to all pages: layouts > application
459421d
@kariabancroft
Copy link

kariabancroft commented Apr 5, 2017

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. When you create the methods in the controllers - separate each method with one like of whitespace plz.
Overall Nice job!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ashtn @kariabancroft