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.

Sign up for GitHub

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

Jump to bottom

Parse telemetry ini config as boolean and make telemetry opt-in. #553

Merged
merged 1 commit into from
Jan 5, 2023
Merged

Parse telemetry ini config as boolean and make telemetry opt-in. #553

merged 1 commit into from
Jan 5, 2023

Commits on Dec 13, 2022

  1. Parse telemetry ini config as boolean and make telemetry opt-in. 

    In issue MisterTea#504, it was brought to attention that while the command line
arg for enabling telemetry was boolean, the ini config in et.cfg needed
to be an int to change the value.  SimpleIni has a GetBoolValue function
that treats the strings "0", "off", "no", "false" as boolean false and
"1", "on", "true" as boolean true.

Also, this will make telemetry opt-in as opposed to opt-out as has been
requested by multiple users.  The supplied et.cfg file will still have
telemetry on.
    @jshort
    jshort committed Dec 13, 2022
    Configuration menu
    Copy the full SHA
    22d970e View commit details
    Browse the repository at this point in the history