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

[FEAT] New service: webex #68

Open
kikanq opened this issue Jan 13, 2024 · 8 comments
Open

[FEAT] New service: webex #68

kikanq opened this issue Jan 13, 2024 · 8 comments
Assignees
Labels
enhancement New feature or request

Comments

@kikanq
Copy link

kikanq commented Jan 13, 2024

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

@kikanq kikanq added the enhancement New feature or request label Jan 13, 2024
@tigattack tigattack changed the title [FEAT] New service: weebex [FEAT] New service: webex Jan 13, 2024
@tigattack
Copy link
Owner

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.

@kikanq kikanq closed this as completed Jan 14, 2024
@kikanq
Copy link
Author

kikanq commented Jan 14, 2024

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?

@kikanq kikanq reopened this Jan 14, 2024
@kikanq
Copy link
Author

kikanq commented Jan 14, 2024

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"}

@kikanq
Copy link
Author

kikanq commented Jan 14, 2024

@tigattack
Copy link
Owner

Does the next step require improving the code for the weebex application?

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.

@kikanq
Copy link
Author

kikanq commented Jan 14, 2024

thanks for the information. when do you expect the merge to be completed?

@tigattack
Copy link
Owner

Difficult to say, sorry. It's currently pending.

@kikanq
Copy link
Author

kikanq commented Jan 16, 2024

Hello,
I created basic notification scripts for weebex. Maybe they will be helpful to you when updating your extensive tool.

https://github.com/kikanq/Veeamnotify_webex/tree/main

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants