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

Long received SMS issue and other recommendations #18

Open
urbanbur opened this issue Sep 22, 2024 · 7 comments
Open

Long received SMS issue and other recommendations #18

urbanbur opened this issue Sep 22, 2024 · 7 comments

Comments

@urbanbur
Copy link

Hi,
first, this is an awesome integration
Been using it since the first version and am now moving to custom_components.

The Issue:
My operator sends a really long notification SMS "Obvescamo vas, da ste presegli 80% dnevnega prenosa podatkov po najvisji razpolozljivi hitrosti v vasem narocniskem paketu, zato bo hitrost po celotni porabi zmanjsana na 64/64 kbps. Za nadaljnjo uporabo podatkovne storitve pri najvisji razpolozljivi hitrosti, ki ne predstavlja dodatnih stroskov, posljite SMS s kljucno besedo NADALJUJ na stevilko 7070. stevilo podaljsanj ni omejeno. Vas Telemach"
On reception, sensor.last_sms becomes unknown.
This script can only receive SMSs up to 254 characters.
I am using MC889.

Minor issue: In Integration, the link for "Known issues" points to unexisting:
https://github.com/Vaskivskyi/ZTE-MC-Home-assistant-repo/issues
instead of your repo.

Recommendations:
Make phone number and message for Send SMS (50GB) customizable.
(it would be even more benefitial if integration could send arbitrary messages to arbitrary numbers for other notifications.
With my operator we have daily, not monthly limit.

@Kajkac
Copy link
Owner

Kajkac commented Oct 8, 2024

Can you try to test this version ?

https://github.com/Kajkac/ZTE-MC-Home-assistant-repo/releases/tag/v1.0.29

  1. Added Last sms to atributes (254 characters and more)
  2. Switched SMS number and message for buttons to configuration
  3. Added 2 more buttons for preconfigured sms sending inside configuration

@urbanbur
Copy link
Author

urbanbur commented Oct 8, 2024 via email

@Kajkac
Copy link
Owner

Kajkac commented Oct 8, 2024

Integration itself is now in prerealease at least this version, techicly speaking it should be easy to delete it and add it again, when i complete whole set of features then updates should work fine, adding a new funcionalitys require delete and re add. (Lots of code refactoring it messes up a lot of things)

and yeah will make iz general, first ever edition was out of hobby for my router in Croatia so thats why it was adjusted like that :)

@urbanbur
Copy link
Author

urbanbur commented Oct 10, 2024 via email

@MiG-41
Copy link

MiG-41 commented Oct 10, 2024

If automation is enabled , also after HA restart is enabled.

@MiG-41
Copy link

MiG-41 commented Oct 13, 2024

@Kajkac ,so now in 1.0.29 ,if there is no any SMS ,(all delated) , entity Last SMS is 999 ...
Previously it was "No SMS" ,or something like that....
And when i sent to router long SMS , i got value now 21 ... (before it was text of sms)
image

@ben33880
Copy link

ben33880 commented Nov 3, 2024

Hi,
entity sensor.recieved_sms_s is never refreshed, I wanted to use this entity to trigger automation upon receipt of SMS

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

4 participants