Skip to content

Conversation

@isabeldepapel
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 Providing the business logic and data for the web app
Describe in your own words what the Controller is doing in Rails Negotiating between the model and the view so that it picks the appropriate method to retrieve/update data in the model using input received in the view
Describe in your own words what the View is doing in Rails Providing the user interface (i.e. how does the web app display on a computer/phone screen?)
What is the purpose of using strong params? (i.e. the params method in the controller) Allows the user to update only those params you specifically mention--keeps user from (un)intentionally changing data that could break your database
How are Rails migrations related to Rails models? They change the schema of the database, which is a part of the model (I think?)
Describe one area of Rails that are still unclear on The model since we didn't really use it. My model is still empty--and what goes in the controller vs the model? For data validation--does that take place in the controller or the model?

@CheezItMan
Copy link

Task List

What We're Looking For

Feature Feedback
Baseline This is fine...
Appropriate Git Usage with no extraneous files checked in Not very many commits and the messages deal mostly with waves completed. I would make the commit messages talk about functionality completed.
Answered comprehension questions Check, The schema is the structure of the database which the model interacts with.
Successfully handles: Index, Show Check
Successfully handles: New, Create Check
Successfully handles: Edit, Update Check
Successfully handles: Destroy, Task Complete Check
Routes follow RESTful conventions Check
Uses named routes (like _path) Check
Overall Very nice, the image made me chuckle! You hit all the requirement. Good work!

Copy link

@CheezItMan CheezItMan left a comment

Choose a reason for hiding this comment

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

Just one comment.

def edit
@task = Task.find_by( id: params[:id].to_i )

unless @task

Choose a reason for hiding this comment

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

Very nice!

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