tuenti / SmsRadar

Android library created to listen incoming and outgoing SMS's
Apache License 2.0
392 stars 116 forks source link

Sms receive observer doesn't work on Android 5.0.1 #5

Open ssaurel opened 9 years ago

ssaurel commented 9 years ago

Hello,

I just tried to use SmsRadar on a project. It works very well on Android 4.4.2 and below.

With device of my wife recently updated with Android 5.0.1, I tried SmsRadar on this Android version. Observe sent SMS works always well.

But, there is an issue to observe received SMS. It doesn't work in Android 5.0.1 .

Do you have experimented the same problem ?

Sylvain

pedrovgs commented 9 years ago

Sorry for the delay, I forgot to answer you. I'll take a look to this issue ASAP. Thanks!

ssaurel commented 9 years ago

Don't hesitate to tell me if you need more help to fix the problem.

ssaurel commented 9 years ago

Some progress ?

pedrovgs commented 9 years ago

Sorry but other tasks keep me busy and I do not think I will be able to devote time to maintaining this lib. Feel free to fork or submit patches, anyway.

vaslabs commented 8 years ago

Hello, thanks for this awesome library. I moved it to Android studio, I had to create a new project and copy paste everything in the end because the maven2gradle wasn't working. I made a few upgrades and tested it for android 5 and 6. It seems to be working now.

Let me know if you want to merge my changes and on what terms (the packages have been renamed because of creating a new project) https://github.com/vaslabs/SmsRadar

Thanks again.

pedrovgs commented 8 years ago

@jplana @oscarsj what's going on with this repository? Don't you have Android developers watching it and reviewing issues and PRs?

dpastor commented 8 years ago

Hi @vaslabs ! Thanks a lot for the work done on this. Let us review your changes and we will get back to you shortly. Thanks a lot again!

vaslabs commented 8 years ago

Hey no problem. Let me know if you need anything on that. I was playing around mostly trying to set it up in Android studio. I haven't touched the source code really, I just updated some dependencies and it needed changes for only the test code. Helped me a lot on that project https://github.com/vaslabs/trackpa_receiver

gmerinojimenez commented 8 years ago

Hello @vaslabs, thanks for the work, your repository seems nice. If you're interested into update this library, please feel free to send us a pull request. But please, keep in mind that we can't merge changes like renaming the library or things like that.

Best regards

vaslabs commented 8 years ago

Thanks. Yeah, I understand. I'll create a branch to accommodate the rename change and send you a pull request :)

Aditya94A commented 8 years ago

Any update on this? Perhaps a warning should be added to the README since this is pretty important.