Closed SilkyPants closed 2 years ago
please merge this 😆
@rodydavis sorry not sure who to ping on this but you were the last one to merge a PR in this repo if you could PTAL 🙏
@rodydavis Commenting to say this would be amazing to have, and you seem like the one to talk to to get things merged around here. Any chance we can get this feature soon?
Reviewing now
Does sending directly on android require the permission in the manifest?
In other words can it be opted out of?
I have had apps rejected because I used the API and wasn't considered a phone messaging app
@rodydavis yes it does require that permission - we've had apps rejected ourselves for the same reason - the category for apps that are allowed to do this is very narrow.
I've made the default for this argument false
and added a warning to the readme about using the feature.
The feature is more "opt-in" than needing to "opt-out" as if you do not add that manifest property and never call with sendDirect: true
it is never used.
Ok that is wonderful to hear! I will merge this and publish soon!
I just wanted to make sure we don't have a lot of apps rejected over a version bump.
Published! https://pub.dev/packages/flutter_sms
Addresses issue #57 to allow the user to send an SMS message directly