Untitled #34

Merged
1 commit merged into from Nov 18, 2010

Conversation

Projects
None yet
2 participants
@raine

raine commented Nov 15, 2010

https://github.com/github/github-services/issues#issue/15

It's an old commit, so I just rebased it on top of the current HEAD so that the merge is less painful.

@technoweenie

This comment has been minimized.

Show comment Hide comment
@technoweenie

technoweenie Nov 18, 2010

Contributor

Added! Please check it out and confirm it works. Thanks for your patience.

Contributor

technoweenie commented Nov 18, 2010

Added! Please check it out and confirm it works. Thanks for your patience.

@raine

This comment has been minimized.

Show comment Hide comment
@raine

raine Nov 19, 2010

It works on pushes, but the Test Hook button doesn't do anything, or at least we're not getting anything in our logs. What's wrong?

raine commented Nov 19, 2010

It works on pushes, but the Test Hook button doesn't do anything, or at least we're not getting anything in our logs. What's wrong?

@technoweenie

This comment has been minimized.

Show comment Hide comment
@technoweenie

technoweenie Nov 19, 2010

Contributor

I don't know, they both use the same mechanism to send out pushes. You should get a push with the last commit... Do you have a test account I can play with?

Contributor

technoweenie commented Nov 19, 2010

I don't know, they both use the same mechanism to send out pushes. You should get a push with the last commit... Do you have a test account I can play with?

@technoweenie

This comment has been minimized.

Show comment Hide comment
@technoweenie

technoweenie Nov 19, 2010

Contributor

Ah, the service hook used api_token, but the docs said token. I fixed the service hook and redeployed.

Contributor

technoweenie commented Nov 19, 2010

Ah, the service hook used api_token, but the docs said token. I fixed the service hook and redeployed.

@raine

This comment has been minimized.

Show comment Hide comment
@raine

raine Nov 19, 2010

It works now. Thanks for all your help. :)

raine commented Nov 19, 2010

It works now. Thanks for all your help. :)

This issue was closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment