Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

Make WebhookMessageEventGrid compatible with the event grid format #3286

Merged
merged 4 commits into from
Jul 12, 2023

Conversation

chkeita
Copy link
Contributor

@chkeita chkeita commented Jul 11, 2023

Summary of the Pull Request

Updated WebhookMessageEventGrid
Updated the dataVersion field to 2.0

closes #3285

@chkeita chkeita requested a review from stishkin July 11, 2023 21:07
@codecov-commenter
Copy link

codecov-commenter commented Jul 11, 2023

Codecov Report

Merging #3286 (cdaaabc) into main (d5b2a33) will decrease coverage by 0.02%.
The diff coverage is 25.00%.

@@            Coverage Diff             @@
##             main    #3286      +/-   ##
==========================================
- Coverage   29.66%   29.65%   -0.02%     
==========================================
  Files         333      333              
  Lines       39377    39377              
==========================================
- Hits        11680    11676       -4     
- Misses      27697    27701       +4     
Impacted Files Coverage Δ
src/ApiService/Tests/OrmModelsTest.cs 0.00% <0.00%> (ø)
...Service/ApiService/onefuzzlib/WebhookOperations.cs 35.71% <66.66%> (ø)
src/ApiService/ApiService/OneFuzzTypes/Webhooks.cs 100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

Copy link
Contributor

@stishkin stishkin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Assuming that you tested this

@chkeita chkeita self-assigned this Jul 12, 2023
@chkeita chkeita enabled auto-merge (squash) July 12, 2023 00:52
@chkeita chkeita merged commit 0226db2 into microsoft:main Jul 12, 2023
@AdamL-Microsoft AdamL-Microsoft mentioned this pull request Jul 13, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

WebhookMessageEventGrid is not compatible with the eventgrid message schema
3 participants