Skip to content

Water - Jessica#53

Open
jwinchan wants to merge 15 commits intoAda-C14:masterfrom
jwinchan:master
Open

Water - Jessica#53
jwinchan wants to merge 15 commits intoAda-C14:masterfrom
jwinchan:master

Conversation

@jwinchan
Copy link

@jwinchan jwinchan commented Nov 3, 2020

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 handles all the data in the database
Describe in your own words what the Controller is doing in Rails The Controller compiles the information from the Model's database and passes that information to the View
Describe in your own words what the View is doing in Rails The View receives the information from the Controller and presents the info in the form of HTML to the user
Describe an edge-case controller test you wrote I wrote tests to make sure that any actions performed on a non-existent task would be redirected instead.
What is the purpose of using strong params? (i.e. the params method in the controller) It dries up the code so that we don't have to spell out every param for a create or update method.
How are Rails migrations related to Rails models? Rails migrations changes the database tables. You can add columns, change types, etc.
Describe one area of Rails that are still unclear on I'm still confused about the term "Convention over Configuration"

Copy link

@dHelmgren dHelmgren left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Task List

Major Learning Goals/Code Review

Criteria yes/no, and optionally any details/lines of code to reference
At least 6 commits with meaningful commit messages ✔️
Routes follow RESTful conventions ✔️
Uses named routes (like _path) ✔️
Creates Models and migrations ✔️
Creates styled views no styling!
Handles errors like nonexistant tasks ✔️
Uses form_with to render forms in Rails ✔️

Functional Requirements/Manual Testing

Functional Requirement yes/no
Successfully handles index & show ✔️
index & show tests pass ✔️
Successfully handles: New, Create ✔️
New, Create tests pass ✔️
Successfully handles: Edit, Update ✔️
Edit, Update tests pass with valid & invalid task ids ✔️
Successfully handles: Destroy, Task Complete ✔️
Tests for Destroy & Task Complete include tests for valid and invalid task ids ✔️

Overall Feedback

Overall Feedback Criteria yes/no
Green (Meets/Exceeds Standards) 5+ in Code Review && 6+ in Functional Requirements ✔️
Yellow (Approaches Standards) 3+ in Code Review && 5+ in Functional Requirements, or the instructor judges that this project needs special attention
Red (Not at Standard) 0-2 in Code Review or 0-4 in Functional Reqs, or assignment is breaking/doesn’t run with less than 5 minutes of debugging, or the instructor judges that this project needs special attention

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