Skip to content

Conversation

@botrethewey
Copy link

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 contains the logical structure of the data and has access to the application's database
Describe in your own words what the Controller is doing in Rails Controller connects the View and the Model, it contains the actions which manipulates the Model
Describe in your own words what the View is doing in Rails View contains the user interface of the application and reflects the Model
What is the purpose of using strong params? (i.e. the params method in the controller) Using strong params ensures that no malicious values is entered into the data base, by specifying keys allowed from the params hash.
How are Rails migrations related to Rails models? Rails migrations are used to manipulate the database schema in the Model
Describe one area of Rails that are still unclear on I am still working on correctly declaring paths in routes and using forms and partial views.

@kariabancroft
Copy link

Task List

What We're Looking For

Feature Feedback
Baseline
Appropriate Git Usage with no extraneous files checked in No. One commit per wave is not enough frequency.
Answered comprehension questions Yes
Successfully handles: Index, Show Yes. Don't forget to add quotes around your class and id values in the HTML. When you use a conditional in your HTML view, you will want to indent the contents inside of that conditional (like in the show view completed logic).
Successfully handles: New, Create Yes. Nice job using the view partial. Class names should be separated by the - character rather than the _ character.
Successfully handles: Edit, Update Yes
Successfully handles: Destroy, Task Complete Yes. In the future you might be able to remove the need for the completed boolean and instead you could calculate that boolean from the information in the date field in the Model.
Routes follow RESTful conventions Yes - I like the way you grouped them together logically using whitespace
Uses named routes (like _path) Yes
Overall Nice job overall! Most of my comments are related to the CSS pieces you added.

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