Skip to content
This repository has been archived by the owner on Oct 15, 2024. It is now read-only.

send SMS method return is still false #18

Closed
nexus05 opened this issue Apr 26, 2017 · 8 comments
Closed

send SMS method return is still false #18

nexus05 opened this issue Apr 26, 2017 · 8 comments

Comments

@nexus05
Copy link

nexus05 commented Apr 26, 2017

Hello...
In #line39 of SIM900.h I changed DEFAULT_INTERCHAR_TIMEOUT to 5000 or 10000
but it still doesn't work...return is always false, even if SMS is not sent.
does anyone knows why?

@lanselambor
Copy link
Contributor

Has fixed this issue, see GPRS_Shield_Arduino.cpp #151.

@ortegafernando
Copy link
Collaborator

Sorry @lanselambor but what do you change is very risky, you are waiting 10000 seconds !!!!

@ortegafernando
Copy link
Collaborator

I have been using this library for months with no problem. May be, @nexus05 could try your proposas with 10 seconds or 20 seconds, but it is not good for anyone to let it with 10000 seconds.

@lanselambor
Copy link
Contributor

@ortegafernando It's not about the timeout parameter, it's for network that before registration SMS can't be send, so in the initialization part isNetworkRegistered() should be invoked.

@ortegafernando
Copy link
Collaborator

@lanselambor Please, rewrite then your proporsal, thanks.

@lanselambor
Copy link
Contributor

@ortegafernando Already there with commit on 21 Nov 2017.

@ortegafernando
Copy link
Collaborator

You have write "10000, 5000", so you will wait 10.000 seconds. Is this ok for you ???? Not for me.

@lanselambor
Copy link
Contributor

Thanks to @ortegafernando PR on Dec 28, 2017, correct timeout parameter error.
While using SIM900 or SIM800, wait for network registration before operating call, sms.

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

No branches or pull requests

3 participants