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

Leaves - Katie K #49

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

Conversation

KKennedyCodes
Copy link

Task List

Updated Note:
Hey Becca, I still can't get my tests to run. My program doesn't like my completed data. The program functions just fine, but the tests are not passing.
Katie

A Note About My Submission:

BECCA! I wasn't able to complete this project. I tried to start over to better understand the process of setting up and creating this project, but when I did that I had conflicting git and databases, as well as, I messed up my rake. I went ahead and did the Ada Zoo project, in hopes to show that I was able to meet the project outcomes. Ada Zoo Link: [https://github.com/KKennedyCodes/rails-zoo]. I'm hoping to get some help to get back to zero and I can attempt again. Sorry for the trouble. - Katie

Comprehension Questions

Question Answer
Describe in your own words what the Model is doing in Rails The model is interacting with the database and gathering the information requested by the controller.
Describe in your own words what the Controller is doing in Rails The controller takes the verb/path given by the router and applies that information to the model and then to the view.
Describe in your own words what the View is doing in Rails The view organizes the data and information on the screen for the user to interact with.
Describe an edge-case controller test you wrote An edge case would be if something was nil or if a page with invalid data was trying to be accessed. These edge cases were to send back an error code and redirect the user.
What is the purpose of using strong params? (i.e. the params method in the controller) Strong params list the variables that the user can interact with and change/update. This keeps users from adding new data variables or new columns to our database.
How are Rails migrations related to Rails models? Migrations change and update the model.
Describe one area of Rails that are still unclear on How to drop and delete databases and migrations. I thought that I deleted my database, but when I made my new project it still accessed migrations to another project.

@beccaelenzil
Copy link

Task List

What We're Looking For

Feature Feedback
Baseline
Appropriate Git Usage with no extraneous files checked in check
Answered comprehension questions check
Successfully handles: Index, Show show is failing because of a typo in the view
Index & Show tests pass check
Successfully handles: New, Create check
New & Create tests pass check
Successfully handles: Edit, Update check
Tests for Edit & Update test valid & invalid task ids check
Successfully handles: Destroy, Task Complete check
Tests for Destroy & Task Complete include tests for valid and invalid task ids see comment
Routes follow RESTful conventions check
Uses named routes (like _path) check
Overall Good job overall on Task List! You hit all of the learning goals: using partials, using strong params, and following Rails conventions. As you already know, your tests are having some issues. Also, make sure you are fully testing for valid and invalid test cases. Finally, do a final check of your website to make sure all your views are working -- typos are easy to fix!

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