Skip to content

Conversation

@SaraChandler
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 is connected to the database. It contains the data.
Describe in your own words what the Controller is doing in Rails The Controller works with the view and the model. It takes in information from the view, interacts with the model and relays changes back to the view.
Describe in your own words what the View is doing in Rails The View creates the interface for the program with HTML, CSS.
What is the purpose of using strong params? (i.e. the params method in the controller) Strong params sanitize user input to conform to certain requirements.
How are Rails migrations related to Rails models? Migrations modify the database schema contained in the model.
Describe one area of Rails that are still unclear on How can we modify models further?

@tildeee
Copy link

tildeee commented Oct 6, 2017

Task List

What We're Looking For

Feature Feedback
Baseline
Appropriate Git Usage with no extraneous files checked in x
Answered comprehension questions x
Successfully handles: Index, Show show page incomplete
Successfully handles: New, Create x
Successfully handles: Edit, Update doesn't work
Successfully handles: Destroy, Task Complete x
Routes follow RESTful conventions x
Uses named routes (like _path) x
Overall

@tildeee
Copy link

tildeee commented Oct 6, 2017

Your editing/updating doesn't work: the controller is trying to use a method called task_params which you haven't defined yet.

Your show page is basically empty!

You didn't get too far with styling on this project.

Good work otherwise making the actions index, new, and delete work

task = Task.find_by(id: params[:id])
redirect_to root_path unless task

if task.update_attributes task_params
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You haven't defined task_params yet

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