Skip to content

Sophie - Earth - Task List#41

Open
steve-messing wants to merge 13 commits intoAda-C14:masterfrom
steve-messing:master
Open

Sophie - Earth - Task List#41
steve-messing wants to merge 13 commits intoAda-C14:masterfrom
steve-messing:master

Conversation

@steve-messing
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 talks to the database and uses schema.rb to define the structure of the database. We haven't done much with the model in this project.
Describe in your own words what the Controller is doing in Rails The controller keeps track of all the methods, logic and functionalities of the project, as defined mainly by our RESTful route paths.
Describe in your own words what the View is doing in Rails The views are HTML files that determine what the user/client sees and interacts with on the web page.
Describe an edge-case controller test you wrote "will respond with redirect when attempting to edit a page with a nonexistent ID" will expect to redirect to the root page if given a path id of -1.
What is the purpose of using strong params? (i.e. the params method in the controller) These refer to the params passed to the controller, and using strong params is a protective measure to prevent end users from passing in their own params and doing creepy stuff.
How are Rails migrations related to Rails models? A migration will integrate changes to the database structure.
Describe one area of Rails that are still unclear on What models can do fo us.

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 ✔️
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