search

go-playground / webhooks

:fishing_pole_and_fish: Webhook receiver for GitHub, Bitbucket, GitLab, Gogs
MIT License
956 stars 237 forks source link

Gitlab Merge Request Assignees #101

Closed lsascha closed 1 year ago

lsascha commented 4 years ago

It seems that gitlab changed the place of asignees in the payload. At least in my tests they are not in the ObjectAttributes.Assignee I had to add an array of Assignee to the MergeRequestEventPayload to get them.

see my diff https://github.com/lsascha/webhooks/commit/122aaa16423b41a7cbc31f3d69c8a55f93ef0b2a.diff

lsascha commented 4 years ago

Also i noticed that the Assignees also contain an email field.

So maybe also add

Email string `json:"email"`

to the Assignee struct if we are at it?