Skip to content

Conversation

@jchiare
Copy link
Contributor

@jchiare jchiare commented May 28, 2018

Reason for this: Had a customer ticket where they had inquiries about this file: https://github.com/PagerDuty/API_Python_Examples/blob/master/EVENTS_API_v2/trigger/trigger_with_incident_key.py

Felt there was some improvements I could help with.

Overview of what I did:

  1. I saw some refactoring I could help with. Big one IMO is that I removed the need to enter the subdomain / api access key for Events API.
  2. I deleted the REST V1 API files due to the decommission.
  3. I renamed INTEGRATION API to EVENTS V1 API.

@jchiare jchiare changed the title Jay2 refactor Jay2 refactor DOGE-180 May 28, 2018
@jchiare jchiare changed the title Jay2 refactor DOGE-180 Jay2 refactor DOGE-149 May 28, 2018
@ladyfractal
Copy link
Contributor

Jay did yeoman's work! Merging the pull request.

@ladyfractal ladyfractal merged commit ffe6797 into master Jun 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants