Custom Properties (`custom_property` and `custom_property_values` events)

colbylwilliams commented 2 months ago

Related to issue #416 and PR #417

Before the change?

initial support for custom_property and custom_property_values (https://github.com/octokit/webhooks.net/pull/417) events was added during preview of Custom Properties and doesn't match the GA'd event payloads.

After the change?

Includes test (GA'd) payloads for both custom_property and custom_property_values events
Updates the models for both events to correctly deserialize the event payloads
Adds custom_property and custom_property_values events to app events (subscriptions)
Adds organization_custom_properties and repository_custom_properties app permissions
Adds custom_properties field to Repository

Pull request checklist

[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been reviewed and added / updated if needed (for bug fixes / features)

Does this introduce a breaking change?

Please see our docs on breaking changes to help!

[ ] Yes
[x] No

github-actions[bot] commented 2 months ago

👋 Hi! Thank you for this contribution! Just to let you know, our GitHub SDK team does a round of issue and PR reviews twice a week, every Monday and Friday! We have a process in place for prioritizing and responding to your input. Because you are a part of this community please feel free to comment, add to, or pick up any issues/PRs that are labled with Status: Up for grabs. You & others like you are the reason all of this works! So thank you & happy coding! 🚀

JamieMagee commented 1 month ago

Thanks for the PR. This change is included in v2.2.1.

octokit / webhooks.net