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

Retire old G5 settings #3530

Merged
merged 1 commit into from
Jun 25, 2024

Conversation

Navid200
Copy link
Collaborator

@Navid200 Navid200 commented Jun 12, 2024

This PR removes 2 settings and the category that only contains the two.
The settings are titled:
Scan constantly
Force Dex to UI thread

Considering we have retired Android 4, 5 and 6, am I wrong to think that no one should need these two settings any longer?

Screenshot_20240612-130522

The following image shows what the Debug settings page will look like for G7 after this PR.
It will look exactly the same for a G6 with the addition of the battery options at the bottom also present.

Screenshot_20240612-122431

Copy link
Collaborator

@jamorham jamorham left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These settings are used by an old old collector which no-one should be using anymore or they will experience many problems. Those settings were experimental when they were produced. You also can remove the findPreference() lookup for these in Preferences.java as well as those are no longer needed either.

@jamorham jamorham merged commit 8c26da8 into NightscoutFoundation:master Jun 25, 2024
1 check passed
@Navid200 Navid200 deleted the Navid_2024_06_12b branch June 25, 2024 20:58
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.

2 participants