-
Notifications
You must be signed in to change notification settings - Fork 101
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
Send SMS in Android without opening a messaging app #44
base: master
Are you sure you want to change the base?
Conversation
Here's an example project that uses my pull request: |
<manifest xmlns:android="http://schemas.android.com/apk/res/android" | ||
package="com.tkporter.sendsms"> | ||
|
||
<uses-permission android:name="android.permission.SEND_SMS" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would this still prompt users of apps that are not using the direct-send feature to use this permission?
@@ -85,6 +89,11 @@ Provides the phone number recipients to show by default | |||
|
|||
`successTypes` (Array (strings), Andriod only, required) | |||
|
|||
`direct_send` (boolean, optional) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you change this to directSend
to keep with the camelCase pattern?
Please DO NOT merge this. Google store has changed its policy and normal apps are not allowed to send sms in the background but to forward the sms sending to a SMS/Communication type Application Refer to docs |
This PR is useful and direct send is optional. that means if somebody doesn't need those feature can, don't use! |
Using this application we can Send SMS in Android without opening a messaging app |
I've added a new prop to enable sending sms in Android without opening a messaging app.
The change is backward compatible (when not setting the prop to
true
, there is not change from current flow).