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

Jillian - Task List Rails #12

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

Jillian - Task List Rails #12

wants to merge 11 commits into from

Conversation

JBoshart
Copy link

I struggled with this project. I felt fairly confident with waves 1, 2, and even 3. Wave 4 was a challenge to say the least, and this pull request is coming in shy of a few of the requirements from that wave. This is due to incredible frustration with migrating and seeding the database (over the weekend)--even with the docs I felt incredibly out of my depth. After struggling with it for quite some time I got in class help today to get almost everything up and running.

I further struggled with Heroku. After finally getting it up and running, I worked on more of Wave 4, got it working in my local, but after my commit and push Heroku gave me the following error upon trying to load the page: "The page you were looking for doesn't exist. You may have mistyped the address or the page may have moved." NOPE. The logs yielded no guidance, nor was I able to find classmate help.

I'm making this pull request before I throw my computer out a window. This was a hard project and a hard day.

@Hamled Hamled assigned noglows and Hamled and unassigned noglows Apr 26, 2016
redirect_to root_path
end

def mark_completed
Copy link

Choose a reason for hiding this comment

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

This is a really good idea!

@Hamled Hamled assigned CShekta and unassigned Hamled May 4, 2016
group :development do
# Access an IRB console on exception pages or by using <%= console %> in views
gem 'web-console', '~> 2.0'
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! so helpful!

@CShekta
Copy link

CShekta commented May 4, 2016

This project is great! I really enjoyed looking over it! I couldn't find a link to the site, but your images are perfect, so I imagine the site was adorable 🎉

And don't be too hard on yourself about Heroku! It is finicky and learning any kind of dev ops is difficult and very frustrating. Deployment is hard, don't worry about it too much. Your code looks fantastic, and that's what counts 👍

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.

4 participants