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

Set IS_TRAINING_APP to true for unbranded #278

Open
mrjones-plip opened this issue Jun 2, 2022 · 6 comments
Open

Set IS_TRAINING_APP to true for unbranded #278

mrjones-plip opened this issue Jun 2, 2022 · 6 comments
Assignees
Labels
Type: Technical issue Improve something that users won't notice

Comments

@mrjones-plip
Copy link
Collaborator

Describe the issue
A number of deployments use the unbranded app for training. This app can be confused for the production app and there's no easy way to tell which is which.

Describe the improvement you'd like
Now that 1.0.0 has the ability to show a warning and red border, we should add this to the unbranded app.

@mrjones-plip mrjones-plip added the Type: Technical issue Improve something that users won't notice label Jun 2, 2022
@mrjones-plip
Copy link
Collaborator Author

@latin-panda - would this count as a 1.0.1 release? Maybe not because it's just a gradle file change?

@latin-panda
Copy link
Contributor

If you want to publish in PlayStore then I think we need a new version, otherwise it won't let you upload APKs with same version.

@mrjones-plip
Copy link
Collaborator Author

1.0.1 it is then! We for sure want it in the play store ASAP

thanks!

@garethbowen
Copy link
Member

We'll need to check what happens if the app is both "training" and pointing to production. Do you get a yellow and a red border?

@mrjones-plip
Copy link
Collaborator Author

Do you get a yellow and a red border?

I agree we need to check for what happens, but unless the app is totally unusable, I don't think it should block this feature from being released ASAP. Based on my research, I think the common use case is to use the unbranded app for training (*.dev.medicmobile.org) or developing (*.my.local-ip.co).

We have a partner that wants to use the app in training on a in a week or two on a *.dev.medicmobile.org domain and would greatly benefit from this feature being live!

@abbyad
Copy link
Contributor

abbyad commented Jun 15, 2022

Adapting some of my comments from a Slack conversation about this issue...

Although the unbranded app may be used for training, it is not exclusively for training. The IS_TRAINING_APP flag will put the training toast for all users of the unbranded app -- including those using it for partner demos and meetings. It's also possible that an unknown (non-Medic) deployment of CHT is using the unbranded app, although I am not aware of that being the case.

Having the border red for production use of app.medicmobile.org makes sense to me, but I don't understand why the unbranded app should have the IS_TRAINING_APP flag. Can someone provide more clarity on that?

Another option is to change the toast for this flavor to be more generic, like This app is not production use. Either way, we should focus on border color and message for the unbranded app, and the implementation details can be sorted out later.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Technical issue Improve something that users won't notice
Projects
None yet
Development

No branches or pull requests

5 participants