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.

Sign up for GitHub

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

Jump to bottom

Handle missing user referenced by team membership #153

Merged

heimweh merged 1 commit into PagerDuty:master from pdecat:team_membership_404

Sep 17, 2019

Contributor

pdecat commented Sep 17, 2019

Same as https://github.com/terraform-providers/terraform-provider-pagerduty/pull/30 but for team membership.

Without this change, if a user is deleted from Pagerduty, refreshing a team membership fails with the following message:

Error: GET API call to https://api.pagerduty.com/users/PWR**** failed 404 Not Found. Code: 2100, Errors: <nil>, Message: Not Found


          Handle missing team membership

c0fec3e

ghost added the size/XS label

heimweh approved these changes

View reviewed changes

heimweh approved these changes

View reviewed changes

Collaborator

heimweh left a comment

Thank you so much @pdecat 🙏
This LGTM 👍

heimweh merged commit bd09ae5 into PagerDuty:master

pdecat deleted the team_membership_404 branch

September 17, 2019 13:53

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels