-
Notifications
You must be signed in to change notification settings - Fork 243
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improper unmarshalling #232
Milestone
Comments
In fact, I think just adding an Line 445 in ab036b7
So it should be
and not
|
Thanks for pointing this out! Would you mind submitting a PR for the fix? |
Erog38
added a commit
to Erog38/go-pagerduty
that referenced
this issue
Jul 16, 2020
Sure thing @stmcallister ! I opened #233 for review to fix this and a couple other spots. I wasn't sure you watched this repo, I assumed most of this code was generated from the openapi spec or similar. |
Erog38
added a commit
to Erog38/go-pagerduty
that referenced
this issue
Apr 5, 2021
adding array definition to ResponderRequest vet your code kids.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
go-pagerduty/incident.go
Line 429 in ab036b7
Use of this field doesn't unmarshal properly and causes the sdk to not allow adding responders to an event.
https://play.golang.org/p/uDs91_9GXDG
The text was updated successfully, but these errors were encountered: