Page MenuHomePhabricator

Remove Twilio-PHP API external
ClosedPublic

Authored by epriestley on Jan 2 2019, 3:16 AM.
Tags
None
Referenced Files
F19505402: D19938.diff
Fri, Jan 9, 11:28 AM
F19063350: D19938.largetrue.diff
Nov 29 2025, 6:19 PM
F19001601: D19938.diff
Nov 21 2025, 3:47 AM
F18937812: D19938.largetrue.diff
Nov 11 2025, 12:27 AM
F18923293: D19938.id.diff
Nov 9 2025, 4:27 PM
F18904431: D19938.id47611.diff
Nov 8 2025, 9:16 AM
F18860555: D19938.largetrue.diff
Nov 2 2025, 11:56 AM
F18788868: D19938.largetrue.diff
Oct 15 2025, 6:50 AM
Subscribers

Details

Summary

Ref T920. D19937 provides about 100 lines of code which can do essentially everything here; throw out the trillion lines of full external API stuff.

(I am generally not sure why everyone writes API libraries like this instead of like D19937.)

Test Plan

Send SMS messages with D19937, so I don't think we need any of this code anymore. This code is techncially reachable through some pathways like bin/sms, but won't be for long.

Diff Detail

Repository
rP Phabricator
Branch
mfa4
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 21454
Build 29215: Run Core Tests
Build 29214: arc lint + arc unit