pulledover icon indicating copy to clipboard operation
pulledover copied to clipboard

Published 20 hours ago • verified publisher icon mattstauffer

Outgoing messages notifying about a recording are too long

Open mattstauffer opened this issue 7 years ago • 6 comments

Sending as multi-segment SMS.

Would be better to have a short URL to link to each.

pulledover.us/r/508123081203

instead of

https://api.twilio.com/2010-04-01/Accounts/AC1234567890abcd1234567890abcdefgh/Recordings/R1234567890abcde1234567890abcdef1234

mattstauffer avatar Sep 08 '17 13:09 mattstauffer

@mattstauffer Love the idea for this app. Would love to take a look at this one for hacktoberfest if you don't already have something in the works.

kkoppenhaver avatar Oct 17 '17 09:10 kkoppenhaver

@kkoppenhaver I don't --please feel free!

mattstauffer avatar Oct 17 '17 13:10 mattstauffer

@mattstauffer What do you think about using the Twilio RecordingSid as the id for pulledover.us/r/id?

cwt137 avatar May 31 '18 02:05 cwt137

@cwt137 I think that's fine

mattstauffer avatar Jun 22 '18 19:06 mattstauffer

@cwt137 If you want to take a crack at it, go ahead. I've not come up with a way to get it working that I'm happy with and that sounds like a solid direction to go in!

kkoppenhaver avatar Jun 22 '18 20:06 kkoppenhaver

I looked at the codebase a little bit and saw that it is using the 4.x version of the Twilio SDK. This has limited functionality compared to the current 5.x version. This task would be easier to accomplish with a newer version of the SDK. I will work on that and then work on making the recording url shorter.

cwt137 avatar Jun 28 '18 02:06 cwt137