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 search #21

Closed
pjaudiomv opened this issue Feb 19, 2018 · 8 comments
Closed

sms search #21

pjaudiomv opened this issue Feb 19, 2018 · 8 comments

Comments

@pjaudiomv
Copy link
Collaborator

it may be nice for the sms search to automatically search for meetings the next day if no meetings are found for current day, then if no meetings are still found post the There are no other meetings found sms. as its just a quick search this may make sense. though not sure how much sms function would be used anyway

@dgershman
Copy link
Collaborator

what is happening now?

@pjaudiomv
Copy link
Collaborator Author

now if you do a sms search it responds with "There are no other meetings for today" i think around line 40 of meeting-search.php. the sms-gateway does a type 1 meeting search for current day. seems somehow it would need to do a cycle through a type 2 meeting search for upcoming if search results = 0

@pjaudiomv
Copy link
Collaborator Author

weird my new england yap does that which is latest version but i just tried it on my charleston yap and it works fine coming up with monday results. maybe its a problem with new englands bmlt server

@pjaudiomv
Copy link
Collaborator Author

cool i think the sms problem may be isolated to the new england server, i just updated my charleston server to the same exact version and it still works and the new england one doesnt.

@dgershman
Copy link
Collaborator

what was your search criteria for New England SMS lookup?

@pjaudiomv
Copy link
Collaborator Author

i tried multiple different searches but primarily edgartown, ma or 02539, and if i run that query after 7pm depending on day it will come up with nothing i tried multiple different servers as well as bmlt servers too. its weird though cause my south carolina one works properly on crna.orgs bmlt server

@dgershman
Copy link
Collaborator

A change for this has been pushed to master. You can give it a spin.

@dgershman dgershman added this to the in-progress milestone Feb 23, 2018
@pjaudiomv
Copy link
Collaborator Author

this is working great now, thanks.

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

No branches or pull requests

2 participants