Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

not going to voicemail #112

Closed
pjaudiomv opened this issue Jul 24, 2018 · 8 comments
Closed

not going to voicemail #112

pjaudiomv opened this issue Jul 24, 2018 · 8 comments
Labels

Comments

@pjaudiomv
Copy link
Collaborator

Describe the bug
phoneline isnt going to voicemail, i currently don't have a custom voicemail mp3.

What version are you running
2.0.1

my php_error log has this in it
[24-Jul-2018 12:59:56 America/New_York] Dialing voicemail
[24-Jul-2018 12:59:57 America/New_York] PHP Fatal error: Uncaught Twilio\Exceptions\RestException: [HTTP 400] Unable to update record: Url is not a valid URL: /voicemail.php?service_body_id=1&caller_id=843-852-3001&caller_number=+15089391663 in /home/charl767/bmlt/yap/vendor/twilio/sdk/Twilio/Version.php:85
Stack trace:
#0 /home/charl767/bmlt/yap/vendor/twilio/sdk/Twilio/Version.php(133): Twilio\Version->exception(Object(Twilio\Http\Response), 'Unable to updat...')
#1 /home/charl767/bmlt/yap/vendor/twilio/sdk/Twilio/Rest/Api/V2010/Account/CallContext.php(108): Twilio\Version->update('POST', '/Accounts/ACbd6...', Array, Array)
#2 /home/charl767/bmlt/yap/helpline-dialer.php(78): Twilio\Rest\Api\V2010\Account\CallContext->update(Object(Twilio\Values))
#3 {main}
thrown in /home/charl767/bmlt/yap/vendor/twilio/sdk/Twilio/Version.php on line 85

@dgershman
Copy link
Collaborator

url of your yap instance?

@pjaudiomv
Copy link
Collaborator Author

@pjaudiomv
Copy link
Collaborator Author

@dgershman
Copy link
Collaborator

I believe this issue might be related to a bug with using forced caller id. Will have to validate this later.

@pjaudiomv
Copy link
Collaborator Author

yes as I turned that off and its working now, ill just leave it off for now and turn on sms notification.

@dgershman
Copy link
Collaborator

I'll work on a patch this evening.

@dgershman dgershman added the bug label Jul 24, 2018
@pjaudiomv
Copy link
Collaborator Author

thanks!

dgershman pushed a commit that referenced this issue Jul 25, 2018
@pjaudiomv
Copy link
Collaborator Author

this works as expected now, thank you very much. this was on a working helpline so I really appreciate it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants