- Notifications
You must be signed in to change notification settings - Fork1.1k
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Not at all necessary 😄 I can see in some of our own |
This is a recurring issue right? |
You're right, I mistook which payload was being edited. Thanks for the catch,@Shredder121. |
davidcelis commentedApr 24, 2016 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
@stevepeak Sorry for the confusion, I've reverted this PR. The PushEvent payload in the API and the webhook payload don't match. This is unfortunately confusing, but has to remain true for backwards compatibility. There's a note that denotes this inthe docs:
I accidentally mentally flip-flopped which payload you were editing, hence the confusion. Sorry again! |
No worries :) Thank you! |
Uh oh!
There was an error while loading.Please reload this page.
Let me know if an example is necessary. Thanks!