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.

Sign up for GitHub

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

Jump to bottom

Omitting xml types to be compatible with B535-232. #54

Closed
wants to merge 4 commits into from
Closed

Omitting xml types to be compatible with B535-232. #54

wants to merge 4 commits into from

Conversation

larsvinc
Copy link

@larsvinc larsvinc commented Dec 4, 2019

XML-type declarations makes send_sms fail on the B535-232.
Fixes #53
Confirmed working fix.

@Salamek
Copy link
Owner

Salamek commented Dec 4, 2019

@larsvinc THX for PR, i will merge this when i'm sure that this is not breaking compability with other devices... (But that may be while since #50 or if someone confirms this is working for Huawei B310s-22)

@larsvinc
Copy link
Author

larsvinc commented Dec 5, 2019

@Salamek sounds good. Here in Norway, the 525 is mostly replaced with the 535 in most stores. I have tried to help with #50 as best as I can.

@larsvinc larsvinc mentioned this pull request Dec 5, 2019
Closed
@Salamek
Copy link
Owner

Salamek commented Dec 5, 2019

@larsvinc THX for donation, README.md and #50 updated...

@Salamek
Copy link
Owner

Salamek commented Jan 7, 2020

@larsvinc thx for PR, used this fix globally in 789aac9, closing pr...

@Salamek Salamek closed this Jan 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

SMS not working for B535-232
2 participants
@larsvinc @Salamek