twiliodeved / ivr-phone-tree-laravel Goto Github PK

The rails implementation has a really simple response when someone visits the / URL:

http://ivr-phone-tree-rails-2.herokuapp.com/

Can we add something like that to this app? Right now it 404s when someone tries to visit the home page, which could be confusing when a new developer is setting up the app for the first time.

I ran into errors following the README because it told me to run composer update instead of composer install. composer update kept erroring out for me.

I double-checked this with some other PHP experts at Twilio, and when a developer is cloning a repo for the first time, we should ask them to run composer install instead.

Everything else in the IVR worked for me except this selection:

https://github.com/TwilioDevEd/ivr-phone-tree-laravel/blob/master/app/Http/Controllers/IvrController.php#L105

When I call +12013409912 I get the standard Twilio error message.

You can either fix the number you created for this tutorial, or use the ones from the Rails port:

https://github.com/TwilioDevEd/ivr-phone-tree-rails/blob/master/app/controllers/twilio_controller.rb#L47-L52

I spotted a few bootstrap dependencies in the project. Since this app doesn't have a UI, we should remove those to make this repo as simple as possible.

Format README based on new standards

• Format headers
• Add Twilio logo at the beginning.
• Add section "How to Demo?"

I was noticing that Twilio connects to the app fine, but often calls immediately disconnect. I dug into the logs at Twilio and saw that it was being caused by a 403 error at amazonaws: "All access to this object has been disabled" (et-phone.mp3). I guess they rate limit the access to that file.

I'm not very experienced with PHP, but I had some trouble setting up this project because the README didn't mention copying the .env.example file to .env.

Can you add a note about that?

I also had to run php artisan key:generate before the app would work, so maybe we need to add that to the README instructions also.