search

kontent-ai / management-sdk-net

Kontent.ai Management .NET SDK
https://www.nuget.org/packages/Kontent.Ai.Management
MIT License
7 stars 31 forks source link

Add new webhooks and rename legacy webhooks #254

Closed lenkaklimcikova closed 10 months ago

lenkaklimcikova commented 11 months ago

Motivation

Enable customers to use both webhook versions - new and legacy. DEL-4734

Sevitas commented 11 months ago

You have renamed the action enums to names with suffix Enum, please keep it consistent in the namespace Kontent.Ai.Management.Models.Webhooks. There are enums without this suffix. I'd prefer removing those suffixes

codecov[bot] commented 10 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (ec7910b) 89.31% compared to head (1d5d606) 84.93%.

:exclamation: Current head 1d5d606 differs from pull request most recent head 26c9d4b. Consider uploading reports for the commit 26c9d4b to get more accurate results

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #254 +/- ## ========================================== - Coverage 89.31% 84.93% -4.38% ========================================== Files 246 263 +17 Lines 2816 2907 +91 Branches 315 320 +5 ========================================== - Hits 2515 2469 -46 - Misses 298 343 +45 - Partials 3 95 +92 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.