Uh oh!
There was an error while loading. Please reload this page.
Add support for projects_v2 and projects_v2_item webhook events - #2868
Conversation
Codecov Report
@@ Coverage Diff @@## master #2868 +/- ##
=======================================
Coverage 98.06% 98.07% =======================================
Files 138 139 +1 Lines 12327 12346 +19 =======================================
+ Hits 12089 12108 +19
Misses 162 162 Partials 76 76
|
gmlewis
commented
Aug 13, 2023
This PR has significant conflicts with #2865 due to said PR dramatically changing the internals for webhook events (which will hopefully also reduce the maintenance burden and increase the simplicity of future PRs like this one). So I apologize for the timing. We can either move forward with this PR and have more work required in #2865 or we can move forward with #2865 then require changes here. I'm honestly fine either way. Do you have a preference @nikpivkin ? |
evankanderson
commented
Aug 13, 2023
I'm happy either way; I'll be updating my PR tomorrow probably |
gmlewis
left a comment
There was a problem hiding this comment.
Thank you, @nikpivkin !
Just a few minor tweaks, please before proceeding.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
gmlewis
left a comment
There was a problem hiding this comment.
Thank you, @nikpivkin !
LGTM.
Awaiting second LGTM+Approval from any other contributor to this repo before merging.
(Maybe @evankanderson has time so that this can be merged before work is continued on #2865 ?)
evankanderson
commented
Aug 14, 2023
Not sure if my review/approval helps, but LGTM. Double checked that the types seem to match the documentation. |
gmlewis
commented
Aug 14, 2023
Yes it does, as you are a valued contributor to this repo. Thank you, @evankanderson ! |
Closes#2867
Added support for projects_v2 and projects_v2_item webhook events.