-
Notifications
You must be signed in to change notification settings - Fork 22
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
all the stuffs...ooops :/ #16
base: master
Are you sure you want to change the base?
Conversation
Hey there! This is Ricky, I'll be leaving a bunch of comments on your PR. I don't know what the Heroku error means without doing some googling - I'd say check with a TA about it. One quick first comment is a reminder to write meaningful commit messages, and commit even more often than you think you should :) |
- Create a custom route and action for showing a complete list of a Person's tasks, both complete and incomplete. | ||
- The url of this page should look something like `http://localhost:5000/people/1/tasks`. | ||
- Link to this action from a Person's `show` view. | ||
- Deploy your completed app to Heroku. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice job copying the readme over. This'll be helpful if people check out your github.
Nice job! From looking this over, it seems like you understand the big key parts of Rails - routing, controllers, models, and tying it all together. As you practice more in Rails, I promise it'll become easier, and you'll feel more confident. Woo hoo! |
Thank you ^^ Sent from my iPhone
|
Um...so yeah. I had a bit of trouble grasping Rails...and I'm still kind of confused on a bit, but it's getting better. So I really just did the bare basics so I could understand as much as possible, no frills today I'm afraid.
I do want to go back and revisit it tonight or sometime this week to style it bc its not styled at all obviously, so yuck.
Also, having problems with heroku:
remote: ! Push rejected, no Cedar-supported app detected
remote: HINT: This occurs when Heroku cannot detect the buildpack
so.....yeah :/ I can has help?
thanks
llama