Skip to content

Conversation

@unklCharlos
Copy link

@unklCharlos unklCharlos commented Apr 29, 2024

  • Sorts out curl requests to create ticket, extract task id and use task id to update requester custom field.
  • Environment variables should already be configured on Circle.
  • Generically renamed ticket template.
  • removes zd creation

Note:

  • Moved some values to become Circle environment variables.
  • May need to sort out OAuth flow; currently just using a personal token.
  • There is a tad bit more cleanup to do as well. I also noticed some different approaches to how I can handle things like extracting the task id from the response using jq. I didn't know if jq would be available until I saw it in the code elsewhere. This did not work for me so I reverted to what I had working.

@unklCharlos
Copy link
Author

unklCharlos commented May 17, 2024

@rigoucr @edwtorba @generalredneck I could use some help here.

  1. The CircleCi build is no longer working for this PR. Seems to be related to the public key? I’m a doofus!! I had local db settings in the repo. That is what was making the build fail :awesome:
  2. I was having trouble with the OAuth flow and just set up a personal token for Bender for now. I may need someone else to look at this to sort out. Or at least provide some direction.

Copy link
Contributor

@generalredneck generalredneck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I mean this looks pretty solid from a code perspective. I say we start giving it a shot and see where it fails.

@unklCharlos
Copy link
Author

@rigoucr Allan approved the code; I also went ahead and removed the ZD ticket creation.

@dalin-
Copy link
Contributor

dalin- commented Jan 22, 2025

Hi all, If I follow, it looks like this can be merged in and released???

@unklCharlos
Copy link
Author

@dalin- I will defer to @rigoucr I recall there was something he was trying to sort out before merging.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants