Skip to content

Add SMS #4

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

Merged
merged 18 commits into from
May 13, 2020
Merged

Add SMS #4

merged 18 commits into from
May 13, 2020

Conversation

brentru
Copy link
Member

@brentru brentru commented May 13, 2020

This pull request adds SMS functionality for the FONA808:

  • send_sms: sends a sms message to a phone number
  • num_sms: property to query sms messages stored in memory or SIM
  • delete_all_sms: deletes all sms messages on FONA SIM
  • delete_sms: deletes a sms message in provided slot
  • read_sms: reads contents of sms message from a slot, returns phone number and sms message
  • New Example fona_sms.py: sends SMS, queries sms messages on SIM, reads SMS messages

fona_sms.py output

FONA SMS
Connecting to network...
...
Connected to network!
RSSI: -90dB
Sending SMS...
Sending SMS Sent!
19 SMS's on SIM Card
('+SNIP', 'Test')
('140404', 'Sorry, this service is not available.')
('+SNIP', 'Today at ten twenty ')
....
('140404', 'Sorry, this service is not available.')

@brentru brentru requested review from a team and ladyada May 13, 2020 17:54
@@ -340,26 +335,19 @@ def network_status(self):
print("Network status")
if not self._send_parse_reply(b"AT+CREG?", b"+CREG: ", idx=1):
return False
if self._buf == 0:
# Not Registered
if self._buf == 0: # Not Registered
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

change to self._buf in (0, 1, 2, 3...) ??

@brentru brentru merged commit 4317c7d into adafruit:master May 13, 2020
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

Successfully merging this pull request may close these issues.

2 participants