Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Queues - Cynthia Cobb - Task List #41

Open
wants to merge 34 commits into
base: master
Choose a base branch
from

Conversation

cynthiacd
Copy link

@cynthiacd cynthiacd commented Mar 28, 2017

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 is in charge of communicating with the DB - the model can create, update, and destroy instances in the db
Describe in your own words what the Controller is doing in Rails The controller is in charge of figuring out how to handle a response. The controller will call model methods and ask for views to render HTML as needed.
Describe in your own words what the View is doing in Rails View is in charge of rendering the HTML and styles
What is the purpose of using strong params? (i.e. the params method in the controller) You can't trust users and what they will input. Strong params is one part of input validation that makes sure users are provided the wanted parameters - This also helps prevent users from inputing stuff that could destroy/harm the db
How are Rails migrations related to Rails models? Migrations create the db tables. If you generate a model you have to run rails db:migrate to implement that table related to that model
Describe one area of Rails that are still unclear on LOTS!! - I am really confused on when to create routes and controller methods that are not part of the restful routes and main controller actions. I couldn't figure out how to do the completed_at function without using it's own route (that was not restful) and implementing its own controller method - I really wanted to use the controller update method but couldn't get it to work

@droberts-sea
Copy link

Task List

What We're Looking For

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

Looks good!

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