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

'Nearby Devices' permission is not on by default when upgrading to 0.5.7 version of the ColdChain App #303

Closed
marthakeezy opened this issue Jun 3, 2024 · 1 comment · Fixed by #311
Labels
Build Tested: Passed Issue has been build tested with all tests passed. Tester: Maria

Comments

@marthakeezy
Copy link

Describe the bug
A clear and concise description of what the bug is. Screenshots are great!

When upgrading to the newest version of the ColdChain App, all permissions should be setup automatically, and the 'Nearby Devices' permission should be on in order to detect the sensors and allow them to connect to the app.

When upgrading to the 0.5.7 version, I noticed that permission is not on by default, which requires a user to go into Settings and toggle the selection.

0.5.6 Default permissions:

photo_2024-06-04 11 51 21

0.5.7 Default permissions:

photo_2024-06-04 11 51 37

To Reproduce
Steps to reproduce the behavior:

  1. Upgrade from your version of the app to the 0.5.7
  2. Go into the ColdChain App -> Sensor -> see the error (the screen keeps spinning and the app is unable to detect any sensors)
  3. Go into Settings -> Apps -> mSupply ColdChain -> Permissions
  4. See 'Nearby Devices' is in the 'Not Allowed Category'
  5. To test -> toggle the selection on (allow Nearby Devices), then close the CCA and open again -> navigate to Sensors -> note you are able to connect them again

Expected behavior
A clear and concise description of what you expected to happen.

All the permissions necessary to run the app should be on by default :)

Smartphone (please complete the following information):

  • Device: [e.g. iPhone6] - Android Galaxy Tab A8
  • OS: [e.g. iOS8.1] - One UI version 6.0
  • Browser [e.g. stock browser, safari] - N/A
  • Version [e.g. 22] Android Version 14, CCA Version 0.5.7

Additional context
Add any other context about the problem here.

@jmbrunskill jmbrunskill linked a pull request Oct 7, 2024 that will close this issue
@marthakeezy
Copy link
Author

Tested in V0.5.7 - fixed :) Thanks team! ✅

@marthakeezy marthakeezy added Build Tested: Passed Issue has been build tested with all tests passed. Tester: Maria labels Oct 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Build Tested: Passed Issue has been build tested with all tests passed. Tester: Maria
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants