-
-
Notifications
You must be signed in to change notification settings - Fork 197
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
sharing by sms with iOs and iMessage #133
Comments
Hello @tpin85, It appears that IOS use a non-standard parsing for the SMS body (more info here). This should now work with the latest version 2.4.4. Thank you for reporting the error. |
@nicolasbeauvais Tested on 2.4.6 and still an issue. |
Tested on 2.4.7 and still an issue |
Hello @MaeseppTarvo and @marshallino16, I do not own any IOS device to debug this issue, please feel free to open a PR with a fix if you can. |
it should use an & instead of ;
see this SO answer: https://stackoverflow.com/questions/39529099/trying-to-send-sms-in-ios-10-is-sms-protocol-broken/39572784#39572784 I temporally fixed it by overwriting the sms_ios sharer via the networks property (https://github.com/nicolasbeauvais/vue-social-sharing#extending-the-network-list) |
Does this mean you needed to update to version 3.x as that github link suggests? I tried overwriting the sms_ios sharer with version 2.4.7 and I think that doesn't work. However I am not sure I can update to version 3.x since I am not using SSR or Vue 3.x |
@holygekko thank you for the code sample, the latest version as been updated with your suggested change! |
@VictorVermeulen Indeed it was not very clear in the documentation, I updated it. The v3 of Vue Social Sharing is fully compatible with Vue2 and non-SSR projects. But I will keep it in beta until Vue3 is released in case something need to be changed to support it. |
Hi,
Sharing by sms is in trouble on iOs and iMessage. Work fine on android. The 'description' param is push into the recipients phones numbers.
You can see in screenshot the issue in iMessage.
Thanks
The text was updated successfully, but these errors were encountered: