- Notifications
You must be signed in to change notification settings - Fork1.1k
Add missing repository.deployments_url field#1003
Uh oh!
There was an error while loading.Please reload this page.
Conversation
tarebyte commentedMar 15, 2016
Hey@Shredder121, thanks for the pull request! I'm taking some time this week to update all of the webhook payloads so that they are all up to date. I'm going to close this, but we really appreciate you taking the time to make the developer site more accurate for users. I'll ping you in this issue to let you know when those changes go out. |
Shredder121 commentedMar 15, 2016
That's cool with me. and of course I do my part. |
Shredder121 commentedJul 1, 2016
Is there any news yet on when this can be expected? |
SKY51717 commentedAug 8, 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.
MikeMcQuaid commentedAug 8, 2016
CC@tarebyte for thoughts on this. |
Shredder121 commentedAug 19, 2016
@tarebyte a 💰 for your 💭? |
tarebyte commentedAug 22, 2016
Sorry for the delay folks, I meant to get around to updating all of the payloads but just ran out of time. @Shredder121 if you open up another PR I'll merge it. Thanks! |
Shredder121 commentedAug 23, 2016
With updated paloads? Or just the missing An aside note, if someone will eventually update it, I am fine with waiting. |
tarebyte commentedAug 24, 2016
@Shredder121 yeah I'm fine with accepting any kind of updated payload. |
I could only quickly get my hands on a handful of events, but I assume that each payload's elements are identical.