search

cyrusinnovation / CodeTestBotApp

Ember client app for the CodeTestBot application.
MIT License
0 stars 2 forks source link

Roles ajax - very clean branch with no extra tests to see if travis will not break #28

Closed lvgelder closed 10 years ago

lvgelder commented 10 years ago

This gives the ability to add and remove roles on the frontend - prevents having to manually assign recruiter roles in the db!

lvgelder commented 10 years ago

:(

childss commented 10 years ago

To clarify, I don't think it has anything to do with your changes. I'm pretty sure this is first build in which they started failing: https://travis-ci.org/cyrusinnovation/CodeTestBotApp/builds/25104120

lvgelder commented 10 years ago

I know, I just wanted to make sure. Now I am sure.

childss commented 10 years ago

Excellent! I ran 

npm update -g bower
npm update
bower update

And now the tests fail like they do on Travis so I'm going to start debugging.

lvgelder commented 10 years ago

oh interesting - I ran bower update but not npm.

childss commented 10 years ago

Success! Cherry pick the top commit from master to fix.

On Thu, May 15, 2014 at 2:09 PM, Lisa van Gelder notifications@github.com wrote:

oh interesting - I ran bower update but not npm.

Reply to this email directly or view it on GitHub: https://github.com/cyrusinnovation/CodeTestBotApp/pull/28#issuecomment-43244762

lvgelder commented 10 years ago

Yay! So this branch has no tests but does work in the real world. :)