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 receive function unexpected number of commas #1

Open
stebl opened this issue Jan 27, 2018 · 1 comment
Open

SMS receive function unexpected number of commas #1

stebl opened this issue Jan 27, 2018 · 1 comment

Comments

@stebl
Copy link

stebl commented Jan 27, 2018

I came across this with an MTQ_LVW3 and a newer firmware build.

The error my device logs is [WARNING] Expected 4 commas. SMS[0] DATA[+CMGL: 1,"REC UNREAD","5551231234","","18/01/26,20:02:42+00"]. Continuing ...

It seems like it originates from this block.

When I use AT+CMGL="ALL" with a serial interface, I get results like this,

+CMGL: 2,"REC READ","5551231234","","18/01/26,19:52:29+00"
Test message

There are indeed 5 commas, and the extra one in the date-time format is unexpected

@stebl
Copy link
Author

stebl commented Jan 28, 2018

I forked this repo and created a fix for this here.
I branched off the commit our project is currently using and tested it with an LVW3 board successfully.

Happy to create a pull request if it would help this project. My concern is there is no test coverage and I don't have resources to test other board types, so all I can say with confidence is my change worked for our use cases.

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

No branches or pull requests

1 participant