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 - Hyunji Kim - TaskList #44

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

Conversation

ricecakemonster
Copy link

@ricecakemonster ricecakemonster commented Mar 30, 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 It provides data base. Rails uses Active Record. Active Record takes raw data and make it usable in Rails.
Describe in your own words what the Controller is doing in Rails It handles what to do when it gets a command from Router. It works inbetween Router, Model and View. Controller is the one with all the methods.
Describe in your own words what the View is doing in Rails It controls what to show on the brower.
What is the purpose of using strong params? (i.e. the params method in the controller) To prevent bad(?) users from manipulating the code by limiting what can be typed into the program. (security reason)
How are Rails migrations related to Rails models? Migration is the process to make structures of Rails models. (Adding columns...) It also keeps a track of changes of database schema
Describe one area of Rails that are still unclear on Still very confusing. I think I understand the process, but dealing with all different files at the same time is very confusing.

@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 - good improvement here!
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