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

No stop sensor for G7 #3199

Closed

Conversation

Navid200
Copy link
Collaborator

There is no circumstance under which anyone would need to stop a G7 session in progress.

If you disagree, please explain. Why would anyone ever need to stop a G7 session?

Otherwise, please let's remove the (redundant) option.

@Navid200 Navid200 assigned Navid200 and unassigned Navid200 Nov 20, 2023
@Navid200
Copy link
Collaborator Author

Navid200 commented Nov 21, 2023

What if I just don't want to receive readings any more for whatever reason?
Just stop collection in xDrip. Then, you will have no more readings from the session. There is no reason to stop the session just because you don't want to receive any more readings.

What if I open my next device and it happens to have an identical pairing code to my current device?
Then, you are going to have a problem with establishing connectivity even if you stop the current session. Stopping the session on your current device is not going to change anything.
You will need to move the old device out of the reception area of your phone. As soon as you do that, you will be able to use your new device. And if you still have a session in progress on it, it will make no difference.


Please tell me why we need to be able to stop a session on a G7.

@Navid200
Copy link
Collaborator Author

Why do we need to remove this option from xDrip. xDrip is working fine with it as is.

Someone is going to tap on it and after, ask, how can I undo this?!

Please, there is no reason whatsoever anyone would ever need this option. Please let's make xDrip cleaner and simpler by removing unneeded options.

@Navid200 Navid200 marked this pull request as draft November 22, 2023 03:40
@Navid200
Copy link
Collaborator Author

Dexcom app also offers to stop the session.
Even though that still makes no sense, I close this PR.

@Navid200 Navid200 closed this Nov 28, 2023
@Navid200 Navid200 deleted the Navid_2023_11_20 branch May 30, 2024 00:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant