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

Branches- Amal #32

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

Branches- Amal #32

wants to merge 10 commits into from

Conversation

ashassan
Copy link

@ashassan ashassan commented Oct 7, 2019

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 is communicating with the database to get the information that the controller wants.
Describe in your own words what the Controller is doing in Rails it holds the logic and coordinates the interaction between the view and model to do the request from the user.
Describe in your own words what the View is doing in Rails The view holds all the code for the user interface.
Describe an edge-case controller test you wrote I wrote a test for destroy that verified that the method would not delete anything when you try to delete the same book twice.
What is the purpose of using strong params? (i.e. the params method in the controller) It dries up the code and improves the security.
How are Rails migrations related to Rails models? We write a migration to create a rails model and to update it.
Describe one area of Rails that are still unclear on I am still a little confused about models in general and what having multiple models would look like.

@jmaddox19
Copy link

Task List

What We're Looking For

Feature Feedback
Baseline
Appropriate Git Usage with no extraneous files checked in X
Answered comprehension questions X
Successfully handles: Index, Show X
Index & Show tests pass X
Successfully handles: New, Create X
New & Create tests pass X
Successfully handles: Edit, Update X
Tests for Edit & Update test valid & invalid task ids X
Successfully handles: Destroy, Task Complete X
Tests for Destroy & Task Complete include tests for valid and invalid task ids X
Routes follow RESTful conventions X
Uses named routes (like _path) X
Overall Great job!

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