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

sms gateway #9

Closed
pjaudiomv opened this issue Feb 5, 2018 · 4 comments
Closed

sms gateway #9

pjaudiomv opened this issue Feb 5, 2018 · 4 comments
Assignees
Labels
Milestone

Comments

@pjaudiomv
Copy link
Collaborator

pjaudiomv commented Feb 5, 2018

know you just did this last night and you may know about it, but ive been testing it and works great and super quick. I do get a warning in my twilio debugger.

Warning 12200
Invalid content was found starting with element 'Say'. One of '{Sms, Message, Redirect}' is expected
get: http://www.mvana.org/sca/chsyapnew/meeting-search.php

<?xml version="1.0" encoding="UTF-8"?> <Response> <Say voice="woman" language="en">Meeting information found, listing the top 5 results.</Say> <Sms>Hollywood Group of Hope Group Monday 7:30 PM 114 Walker Ln. in Goose Creek, SC</Sms> <Sms>All Kidding Aside Group Monday 7:00 PM 3250 Ladson Road in Ladson, SC</Sms> <Sms>Step Working Study Guide Group Monday 6:00 PM 2010 Hawthorne Dr in North Charleston, SC</Sms> <Sms>Primary Purpose Monday 9:00 AM 2010 Hawthorne Ave in North Charleston, SC</Sms> <Sms>Back to Basics Monday 12:00 PM 2010 Hawthorne Ave. in Bldg in back in North Charleston, SC</Sms> <Say voice="woman" language="en">Thank you for calling, goodbye.</Say> </Response>

I temporary fixed this by creating a meating-searchsms.php and removing those say parts then directing my sms gateway to that, but I but you have some tricks up your sleave. maybe making those parts conditional to only voice calls

@dgershman
Copy link
Collaborator

yeah this is a known issue, thanks for opening this.

@dgershman dgershman added the bug label Feb 5, 2018
@dgershman dgershman added this to the in-progress milestone Feb 6, 2018
@dgershman dgershman self-assigned this Feb 6, 2018
@dgershman
Copy link
Collaborator

fixed in 96d2c3b?diff=split

@dgershman dgershman modified the milestones: in-progress, released Feb 6, 2018
@dgershman
Copy link
Collaborator

please verify

@pjaudiomv
Copy link
Collaborator Author

awesome that fixed it, no warnings and works fine

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