Skip to content

Conversation

@bonara
Copy link

@bonara bonara commented Apr 15, 2019

Task List

Congratulations! You're submitting your assignment!

Comprehension Questions

Question Answer
Describe in your own words what the Model is doing in Rails Model is in charge of the database records.
Describe in your own words what the Controller is doing in Rails Controller is responsible for making sense of the user's request by either interacting with the Model or/and View.
Describe in your own words what the View is doing in Rails View is responsible for compiling a response that controller asks for.
Describe an edge-case controller test you wrote One if the edge-case controller tests I wrote checks if the request returns 404 response when user provides an invalid task ID.
What is the purpose of using strong params? (i.e. the params method in the controller) It provides an interface for protecting attributes from end-user assignment.
How are Rails migrations related to Rails models? Rails migrations make the changes to our database, and update our schema.
Describe one area of Rails that are still unclear on It will take some time to get used to adding custom routes and methods.

@CheezItMan
Copy link

Task List

What We're Looking For

Feature Feedback
Baseline
Appropriate Git Usage with no extraneous files checked in Reasonable number of commits and good commit messages, no extraneous files checked in that I could see.
Answered comprehension questions Check, a view determines the appearance returned to the user. Hopefully you will have a better feel for custom routes now.
Successfully handles: Index, Show Check
Index & Show tests pass Check
Successfully handles: New, Create Check
New & Create tests pass Check
Successfully handles: Edit, Update Check
Tests for Edit & Update test valid & invalid task ids Check
Successfully handles: Destroy, Task Complete Check
Tests for Destroy & Task Complete include tests for valid and invalid task ids Check
Routes follow RESTful conventions Check
Uses named routes (like _path) Check
Overall Nice work, you hit all the learning goals of this project.

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