Live data from Hacker News

Show HN: GitBattle – Battle two Github accounts

gitbattle.com

31–40 of 45 posts

Re: Show HN: GitBattle – Battle two Github accounts

#31
post #30

If this is failing for you, make sure you haven't blocked twitter using Ghostery or the like, as this causes the script to error out. Uncaught ReferenceError: twttr is not defined app.js:196 setupWinner app.js:196 Maybe the script can be made a bit more robust in this case?

Good idea! Added a check to see if twttr is defined, if it isn't I just don't add the share through twitter button. Thanks!

Re: Show HN: GitBattle – Battle two Github accounts

#33

Earlier quoted context omitted.

are there still repos left under the processing box? if so, would you mind checking your console?

Uncaught ReferenceError: twttr is not defined EDIT: My cookie-blocking plugin apparently was blocking something critical.

Also got the same error due to the Twitter button not loading. It might be a good idea to handle that case in the code.

Re: Show HN: GitBattle – Battle two Github accounts

#34
post #30

If this is failing for you, make sure you haven't blocked twitter using Ghostery or the like, as this causes the script to error out. Uncaught ReferenceError: twttr is not defined app.js:196 setupWinner app.js:196 Maybe the script can be made a bit more robust in this case?

Good idea! Added a check to see if twttr is defined, if it isn't I just don't add the share through twitter button. Thanks!

No worries :)

I should have opened a pull request for you!

Awesome to see the quick turn around on your repo, though I haven't seen it deployed yet. It looks like you are using Heroku to deploy, have you not deployed yet or do I need to refresh my client-side resources?

Re: Show HN: GitBattle – Battle two Github accounts

#36
post #34

Earlier quoted context omitted.

Good idea! Added a check to see if twttr is defined, if it isn't I just don't add the share through twitter button. Thanks!

No worries :) I should have opened a pull request for you! Awesome to see the quick turn around on your repo, though I haven't seen it deployed yet. It looks like you are using Heroku to deploy, have you not deployed yet or do I need to refresh my client-side resources?

Made a little mistake, sorry, should be deployed in 10 seconds.

Re: Show HN: GitBattle – Battle two Github accounts

#37

Earlier quoted context omitted.

Uncaught ReferenceError: twttr is not defined EDIT: My cookie-blocking plugin apparently was blocking something critical.

Also got the same error due to the Twitter button not loading. It might be a good idea to handle that case in the code.

should be handled now, sorry about that!

Re: Show HN: GitBattle – Battle two Github accounts

#38
post #34

Earlier quoted context omitted.

No worries :) I should have opened a pull request for you! Awesome to see the quick turn around on your repo, though I haven't seen it deployed yet. It looks like you are using Heroku to deploy, have you not deployed yet or do I need to refresh my client-side resources?

Made a little mistake, sorry, should be deployed in 10 seconds.

Ha, no need to apologise! The fix works great.

Re: Show HN: GitBattle – Battle two Github accounts

#39
Well, this makes my devops decisions no easier as it's pretty much a dead heat:

http://www.gitbattle.com/?u1=puppetlabs&u2=opscode

(incidentally, there appears to be a UI bug where it lists some repos multiple times - at the end the list looked like puppet/chef/puppet/chef/puppet/chef )

Re: Show HN: GitBattle – Battle two Github accounts

#40
Neat! It'd be really cool to see the top-scorer in an organization. Trying to compute the score of tobytripp (a coworker) leaves me at the secret of life with both "Error in event handler for 'undefined': Cannot read property 'qrstr' of undefined TypeError: Cannot read property 'qrstr' of undefined " and "Uncaught SyntaxError: Unexpected token ILLEGAL" in my console.
Post reply on HN