Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat(integrations): Add Twilio API Key integration #3351

Closed
wants to merge 1 commit into from

Conversation

akhazaei
Copy link

Adding Twilio Integration to Nango

Copy link
Collaborator

@bodinsamuel bodinsamuel left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution 🙏🏻
Minor comments, let us know if you want us to take over

Comment on lines +6 to +16
## Features

- Send SMS messages
- Make voice calls
- Manage phone numbers
- Access message history
- Handle webhooks

## API Reference

The Twilio API reference can be found [here](https://www.twilio.com/docs/api).
Copy link
Collaborator

Choose a reason for hiding this comment

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

You can remove that


The Twilio API reference can be found [here](https://www.twilio.com/docs/api).

## Authentication
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
## Authentication
# Overview

1. Account SID
2. Auth Token

## Setup
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
## Setup
### Prerequisites:


## Setup

### Step 1: Find your Account SID and Auth Token
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
### Step 1: Find your Account SID and Auth Token
#### Step 1: Find your Account SID and Auth Token

Keep your Auth Token secure and never share it publicly.
</Tip>

### Step 2: Create Nango Connection
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
### Step 2: Create Nango Connection
#### Step 2: Enter credentials in the Connect UI

Comment on lines +37 to +57

```bash
curl --request POST \
--url https://api.nango.dev/connection \
--header 'Authorization: Bearer <NANGO_SECRET_KEY>' \
--header 'Content-Type: application/json' \
--data '{
"provider_config_key": "twilio",
"connection_id": "my-twilio-connection",
"credentials": {
"apiKey": "<AUTH_TOKEN>"
},
"connection_config": {
"accountSid": "<ACCOUNT_SID>"
}
}'
```

## Rate Limits

Twilio's rate limits vary by account type and API endpoint. Check the [Twilio Rate Limits documentation](https://www.twilio.com/docs/usage/api/rate-limits) for details.
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is not the place for this guide, check other connect.mdx 👍🏻

categories:
- dev-tools
- communication
auth_mode: API_KEY
Copy link
Collaborator

Choose a reason for hiding this comment

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

It seems it's a basic auth, user is the accountSid and apiKey is the password, you can check lastpass provider that has a similar configuration

Copy link
Collaborator

Choose a reason for hiding this comment

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

Unrelated, if you can remove please 👍🏻

@bodinsamuel bodinsamuel requested a review from a team January 24, 2025 10:45
@akhazaei
Copy link
Author

@bodinsamuel it would be great if you guys can take over!! 🙏

@bodinsamuel
Copy link
Collaborator

Was not able to push changes to this PR, reup it there #3393 3393

bodinsamuel added a commit that referenced this pull request Jan 29, 2025
## Changes

Copied from #3351

original code from @akhazaei

- Add support for Twilio
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants