Gitlab Merge Request Assignees #101
Comments
|
Also i noticed that the Assignees also contain an email field. So maybe also add to the Assignee struct if we are at it? |
riyadhalnur
added a commit
to riyadhalnur/webhooks
that referenced
this issue
May 2, 2021
- gitlab updated their MR payload which adds a parent field for the list of assignees - this fix update the Assignee struct to add the missing `Email` and `ID` field - adds an array of assignees as part of the MR event payload Closes go-playground#101
riyadhalnur
added a commit
to riyadhalnur/webhooks
that referenced
this issue
May 2, 2021
- gitlab updated their MR payload which adds a parent field for the list of assignees - this fix update the Assignee struct to add the missing `Email` and `ID` field - adds an array of assignees as part of the MR event payload Closes go-playground#101
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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
The text was updated successfully, but these errors were encountered: