Skip to content
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

Params and users #11

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Conversation

halcyonyarn
Copy link

Hello! This API that you've made is great. The code is awesome. I'm not sure if these changes will be useful for you or others, but figured I'd let you take a look. Also, I'm not sure I've followed your style sufficiently, apologies for the hacks.

In general - I was wanting a couple things: first, to be able to add tickets from our CRM, and add a user if needed. So with these mods, we can first check if the user existed in OST, and if they didn't, add them automatically.

Second, the tickets from this API were not using the 'status' field in some views, and the title was displaying differently from manually created tickets. I added some insert queries to add form table data, where it seems those are stored? The result looks more 'complete' in my environment when reviewing tickets.

Finally, we wanted a way to add internal notes when a ticket is created from our CRM. It's a useful way to get agents up to speed, without having that info viewable. So, I added this via some new "optionalParameters" hacking...

Most other things were minor. Apologies for the 'spacing' changes, it seems my VIM does that automatically...

Thanks again for this project and all the best!

@BMSVieira
Copy link
Owner

Hey,
Im glad it helped you to achieve what you wanted.
I will take a look, thank you!

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.

2 participants