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

Brenna's TaskList #36

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

Brenna's TaskList #36

wants to merge 14 commits into from

Conversation

bcmdarroch
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 is a Ruby class that interacts with the database and provides its data to the Controller.
Describe in your own words what the Controller is doing in Rails The Controller is a Ruby class that acts as the intermediary between the Model and View. It gains access to data from the database through the Model and passes that information to the View, so it can ultimately be displayed to the user.
Describe in your own words what the View is doing in Rails The View is what the user sees. It is written in HTML, CSS, and ERB.
What is the purpose of using strong params? (i.e. the params method in the controller) This is security measure to prevent any code or other malicious content from being passed into the program through user forms.
How are Rails migrations related to Rails models? Migrations create/manipulate the database that the models interact with.
Describe one area of Rails that are still unclear on We didn't really make any changes to the Model file. Is that typical? What changes might we make?

@kariabancroft
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. We will start putting things in the model file next week!
Successfully handles: Index, Show Yes. Nice job using a custom partial to handle the completion logic. You can conditionally apply classes as well as conditionally display data so in the future you could clean up this logic a bit.
Successfully handles: New, Create Yes. Nice job using the form partial. It's not too common to use ul and li to contain form items, but we'll see some other styling options for these later on.
Successfully handles: Edit, Update Yes. Nice job using a local variable in the update controller action.
Successfully handles: Destroy, Task Complete Yes. Nice job using Time.current.
Routes follow RESTful conventions Yes
Uses named routes (like _path) Yes
Overall Very nice job. You incorporated each piece from class and extended beyond using partials.

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