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

Alena's TaskList #29

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

Alena's TaskList #29

wants to merge 17 commits into from

Conversation

Spatterjaaay
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 Model describes the objects that the application is managing and the types of data that are valid for each of the attributes. It also interacts with the database.
Describe in your own words what the Controller is doing in Rails Controller is central part of the application. It interacts with model, receives requests from the web server, and sends information back to views, which then displays the data. It also contains the logic of the application.
Describe in your own words what the View is doing in Rails View generates the interface that displays data to the user. Most commonly View is an html file and since we are using Ruby, .erb extension is added as well.
What is the purpose of using strong params? (i.e. the params method in the controller) We can never pass data into the Model directly, instead we need to specify which keys we allow to be passed form the params hash using strong params. It is a security measure that prevents the injection of harmful data.
How are Rails migrations related to Rails models? Migrations are changes to the database structure (schema). Using migrations we create, structure and change the Model.
Describe one area of Rails that are still unclear on I'm still working on orienting myself in the file structure of the entire application and how individual parts of the application interact with each other. I have also discovered that not all migrations can be rolled back, and I am not sure why.

@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
Successfully handles: Index, Show Yes. Nice job using a custom partial to display the toggle completion button! In the future, you could use the conditional to set a variable and use a single button_to with a variable plugged in rather than duplicating most of the button_to logic.
Successfully handles: New, Create Yes. Nice job using the partial for the form.
Successfully handles: Edit, Update Yes. Nice job using the partial.
Successfully handles: Destroy, Task Complete Yes. I like the way this method toggles the completion.
Routes follow RESTful conventions Yes
Uses named routes (like _path) Yes
Overall Nice job exploring the pieces of the request cycle in this assignment. I like the way you used an additional partial to DRY up your views.

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