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

Add the phone number to outgoing webhooks #5154

Open
gerryd opened this issue Oct 10, 2024 · 2 comments · May be fixed by #5155
Open

Add the phone number to outgoing webhooks #5154

gerryd opened this issue Oct 10, 2024 · 2 comments · May be fixed by #5155

Comments

@gerryd
Copy link

gerryd commented Oct 10, 2024

What would you like to see!

Currently, the payload available to webhooks contains limited serialised user information:

{
    "users_to_be_notified": [
        {
            "id": "ZT8PD2FLQB32A",
            "username": "admin",
            "email": "admin@localhost"
        },
}

At a minimum, it would be useful to add a phone number; example:

{
    "users_to_be_notified": [
        {
            "id": "ZT8PD2FLQB32A",
            "username": "admin",
            "email": "admin@localhost",
            "phone_number": "+1123255445788"
        },
}

Use case: I have an existing notification system which I would like to plug in to. I currently store phone number information in there, I would like to move this information to oncall instead.

For this to be useful, I guess having #2470 would also be nice.

Product Area

Alert Flow & Configuration

Anything else to add?

No response

Copy link
Contributor

The current version of Grafana OnCall, at the time this issue was opened, is v1.10.5. If your issue pertains to an older version of Grafana OnCall, please be sure to list it in the PR description. Thank you 😄!

@gerryd gerryd linked a pull request Oct 10, 2024 that will close this issue
3 tasks
@jarmd
Copy link

jarmd commented Nov 28, 2024

This is something we also need to be able to use own carrier system for SMS.

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

Successfully merging a pull request may close this issue.

2 participants