octokit / webhooks.net

GitHub webhook events toolset for .NET
MIT License
52 stars 32 forks source link

[BUG]: ProcessWorkflowRunWebhookAsync not called for `in_progress` messages #580

Closed justinmchase closed 1 month ago

justinmchase commented 1 month ago

What happened?

The webhook event is not firing for in_progress webhook messages.


Screenshot 2024-09-28 at 9 44 34 AM

But this code appears to filter out all events except for requested and completed...

Screenshot 2024-09-28 at 9 43 07 AM

Versions

dotnet 13.0.1

Relevant log output

No response

Code of Conduct

github-actions[bot] commented 1 month 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 labeled with Status: Up for grabs. You & others like you are the reason all of this works! So thank you & happy coding! 🚀

allantargino commented 1 month ago

Hi @justinmchase, I experimented the same bug in my current project. I was also able to reproduce it in the AspNetCore Sample.

I just send the potential fix (which made the sample work).

justinmchase commented 1 month ago

Thank you, looks perfect.