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

Tasklist App #19

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

Tasklist App #19

wants to merge 33 commits into from

Conversation

meliquintero
Copy link

end

group :development do
gem 'better_errors'
Copy link

Choose a reason for hiding this comment

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

love better_errors! great choice 👍

redirect_to root_path
else
render :new
end
Copy link

Choose a reason for hiding this comment

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

great job handling an error case here!

@CShekta
Copy link

CShekta commented Apr 26, 2016

Great job overall! Loved your site and your design! Heroku can be finicky so I would double check your links! Also for your commit messages, they should be meaningful descriptions of what you're updating. It took me ages to actually start doing that, but it becomes helpful later when you need to go back and see what you changed (or if you're me, when exactly you added something that broke everything else). Keep up the amazing work - you're fantastic!

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