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.

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

Figure out a way to enable SMS or remove all SMS code by September 2019 #2934

Closed
yanokwa opened this issue Mar 4, 2019 · 5 comments · Fixed by #3913
Closed

Figure out a way to enable SMS or remove all SMS code by September 2019 #2934

yanokwa opened this issue Mar 4, 2019 · 5 comments · Fixed by #3913
Assignees
Milestone

Comments

@yanokwa
Copy link
Member

yanokwa commented Mar 4, 2019

The decision has not been made yet, but I'm filing this issue so we don't forget.

Either we find some way to re-enable SMS in a compliant way, or we remove the SMS code. I'd like to make this decision by Sep 2019 (6 months after the policy was in place) because if we can't figure it out by then, we probably won't do it.

Related: #2738

@yanokwa yanokwa changed the title Remove all SMS code by September 2019 Figure out a way to enable SMS or remove all SMS code by September 2019 Mar 4, 2019
@abbyad
Copy link

abbyad commented Jun 26, 2019

There are a couple of options available even without having the permission in the Play store:

  1. Product flavor or build parameter to include the SEND_SMS permission and the related code. You can have different permissions in a separate product flavor. By doing it that way the Collect app in the Play store would not have the SMS features, but builds with and without the SMS features could be made available in the GitHub Releases.

  2. Android Intent to send the SMS with the default SMS app when the SEND_SMS permission is not given. There are some negative UX implications such as requiring the user to click Send from their SMS app, as well as having the content user editable and readable. That said, it could be better than not having SMS at all!

The options are not completely mutually exclusive, and could be implemented together.

An advantage for those who maintain their own branches of Collect would be that they could stay more up to date with the rest of Collect changes while also choosing to include the SMS features - they may even not need their own branch!

What do you think of these options? Would either (or both) seem feasible?

@abbyad
Copy link

abbyad commented Sep 13, 2019

After discussion with @yanokwa and the Medic Mobile team we are exploring the creation of a side loaded app that handles sending SMS via an Intent. This is similar to option 2 above, but instead of relying on the default messaging app, we'd explicitly call a (side-loaded) app whose sole purpose is to relay the SMS.

Note that apps that use this side-loaded SMS app could optionally fall back to sending via the default messaging app if the relaying app is not present. In this case the message content would ideally be converted so that is it not human-readable and contains a checksum to know if it was altered.

We are working to contribute this SMS relay app to the Community Health Toolkit so that community health apps built with medic-android can use it -- and think that it will be a useful way for Collect to maintain its SMS functionality as well. If anyone else has started something similar, or is interested in contributing please respond here, or join the conversation in medic/cht-android#71.

@lognaturel
Copy link
Member

The strategy described above seems like a good one. Has progress been made in that direction?

It would likely require fairly significant changes to the Collect SMS functionality. We keep running into code or resources for SMS that are no longer in use and causing confusion. Let’s remove them for now and it can All easily be retrieved from source control and adapted for a new approach in the future.

@lognaturel lognaturel added this to the v1.27 milestone Apr 6, 2020
@grzesiek2010 grzesiek2010 self-assigned this Apr 21, 2020
@lognaturel lognaturel modified the milestones: v1.27, v1.28 May 12, 2020
@grzesiek2010 grzesiek2010 mentioned this issue Jun 9, 2020
3 tasks
@Supriyo9
Copy link

Hey , still you didn't find any way except removing the code

@lognaturel
Copy link
Member

Because of the way Play Store restrictions now work, there is no plan to bring back SMS functionality to Collect. This documentation and the section it's part of describes how to send an SMS using the default SMS app. This is a sample form.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants