Skip to content

Conversation

@LauraAddams
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 The model interacts with the database, handles business logic.
Describe in your own words what the Controller is doing in Rails The controller communicates between the view and model, it retrieves info from each part before returning the information to the user.
Describe in your own words what the View is doing in Rails The view is purely what the user sees, it determines how everything is presented.
What is the purpose of using strong params? (i.e. the params method in the controller) Strong params prevents users from adding additional params, it's a security measure for the prevention of malicious data being added.
How are Rails migrations related to Rails models? Migrate either creates of changes a model's schema.
Describe one area of Rails that are still unclear on Additional built-in methods would be great to know!

@tildeee
Copy link

tildeee commented Oct 5, 2017

Task List

What We're Looking For

Feature Feedback
Baseline
Appropriate Git Usage with no extraneous files checked in You're aware you didn't commit much!
Answered comprehension questions x
Successfully handles: Index, Show x
Successfully handles: New, Create x
Successfully handles: Edit, Update x
Successfully handles: Destroy, Task Complete x
Routes follow RESTful conventions x
Uses named routes (like _path) x
Overall

end

def create
@task = Task.new(name: params[:task][:name], description: params[:task][:description])
Copy link

Choose a reason for hiding this comment

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

You can actually probably use your task_params method here, too!

@tildeee
Copy link

tildeee commented Oct 5, 2017

Good work overall! Your site looks elegant and meets the requirements. 🎉

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