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

Stacks - Jackie - Task List #31

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

Conversation

jackiewatanabe
Copy link

@jackiewatanabe jackiewatanabe commented Mar 27, 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 contains the objects or data in the database
Describe in your own words what the Controller is doing in Rails The controller dictates what happens with the model objects and browser views based on what the user does.
Describe in your own words what the View is doing in Rails The view is how the webpage will look in the user browser. View only deals with the presentation of data.
What is the purpose of using strong params? (i.e. the params method in the controller) strong params prevents malicious data from being used. It only allows specified data to get through to the database. For example, if a hash were to contain other data than what was allowed, the unspecified data would be ignored and not stored in the database.
How are Rails migrations related to Rails models? migrations allow for changes to models. For example, more attributes for one model (in the form of columns) or data types of those attributes
Describe one area of Rails that are still unclear on all of it...?

…ask_params to prevent hash data other than name, description and completion date from being read in
…ned it's not necessary because it's already in the layouts folder
…and also added classes to links for tasks on index page to change colors based on completion status
@CheezItMan
Copy link

Task List

What We're Looking For

Feature Feedback
Baseline
Appropriate Git Usage with no extraneous files checked in Decent number of commits.
Answered comprehension questions Check, let me know if you have questions on Rail I can answer...
Successfully handles: Index, Show Check
Successfully handles: New, Create Check, Good use of the partial view!
Successfully handles: Edit, Update Check
Successfully handles: Destroy, Task Complete Check
Routes follow RESTful conventions RESTful Routes used
Uses named routes (like _path) Check
Other I did notice that your homepage routes is a little off, `root 'task#index'
should be insteadroot to: 'tasks#index'`
Overall Overall very nice work. Good work covering all the requirements of the project. I like the use of rollover effects on the styling.

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