Skip to content

feat: add Twilio SMS destination - #350

Merged
masukai merged 2 commits into
drt-hub:mainfrom
PFCAaron12:feat/add-Twilio-SMS-destination
Apr 16, 2026
Merged

feat: add Twilio SMS destination#350
masukai merged 2 commits into
drt-hub:mainfrom
PFCAaron12:feat/add-Twilio-SMS-destination

Conversation

@PFCAaron12

Copy link
Copy Markdown
Collaborator

What does this PR do?

  1. drt/destinations/twilio.py — Send SMS via Twilio API

  2. Auth via Account SID + Auth Token (Basic auth)

  3. Jinja2 template for message body
    from_number and to_template (per-row phone number)

  4. Unit tests with mocked HTTP client

  5. Added to drt/cli/main.py

  6. Twilio requires E.164 format for phone numbers so validation for this was added to twilio.py

Related Issue

Closes: # 159

Checklist

  • Tests pass (make test)
  • Linter passes (make lint)
  • Updated CHANGELOG.md (if user-facing change)

@codecov

codecov Bot commented Apr 16, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 90.12346% with 8 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
drt/cli/main.py 0.00% 3 Missing ⚠️
drt/config/models.py 83.33% 3 Missing ⚠️
drt/destinations/twilio.py 96.66% 2 Missing ⚠️

📢 Thoughts on this report? Let us know!

@masukai masukai left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🎉 Clean Twilio implementation — the E.164 validation for both from_number (upfront) and to_number (per-row) is a thoughtful touch that prevents silent delivery failures. Template-based message body + per-row phone targeting is a solid design.

I pushed a small ruff fix (unused import, line length, import ordering) — no action needed from your side.

Three connectors in a day is impressive momentum @PFCAaron12 — great to see this level of engagement! If you enjoy using drt, a ⭐ on the repo would be appreciated 🙏

@masukai
masukai merged commit 5c75dda into drt-hub:main Apr 16, 2026
6 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 16, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants