- Notifications
You must be signed in to change notification settings - Fork1.1k
Add missing repository.deployments_url field#1003
Conversation
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. |
That's cool with me. and of course I do my part. |
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.
CC@tarebyte for thoughts on this. |
@tarebyte a 💰 for your 💭? |
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! |
With updated paloads? Or just the missing An aside note, if someone will eventually update it, I am fine with waiting. |
@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.