-
Notifications
You must be signed in to change notification settings - Fork 7
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
[FEAT] New service: webex #68
Comments
Looking at the Webex incoming webhook docs, it seems to accept Markdown. This will be quite simple to add support for, I'll see if I can look into it once #63 is merged since support for Webex will look very similar to that. Alternatively, if you need this quickly, you're welcome to submit your own pull request with the necessary changes. |
Hi ! I am not a programmer and it will be difficult for me to understand some phrases, so I will write what I managed to do after your answer. I managed to connect weebhook to my company account - I created a room in my webex and completed all the fields in the weebhook integration, and obtained the weebhook address for my connection with webex. Does the next step require improving the code for the weebex application? |
the code below when invoked locally on my laptop works (of course after entering the correct webhook ID). the test message is visible in the weebex area I created.Invoke-RestMethod -Uri "https://webexapis.com/v1/webhooks/incoming/YOUR_WEBHOOK_ID" -Method POST -Body '{"markdown": "Test message"}' -Headers @{"Content-Type"="application/json"} |
Yes that's right. It could be based heavily on the changes in #63, however I would suggest waiting until that PR is merged since the changes aren't yet complete, so you'd be basing your changes on possibly incorrect code. |
thanks for the information. when do you expect the merge to be completed? |
Difficult to say, sorry. It's currently pending. |
Hello, |
Is your feature request related to a problem?
hello, how to integrate it with webex in the company?
Describe your ideal solution
Definition of done
Stretch
Further comments
The text was updated successfully, but these errors were encountered: