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

Activate statistics collection parameter cannot be turned off #595

Closed
1 task done
gelavat opened this issue Jun 29, 2024 · 5 comments
Closed
1 task done

Activate statistics collection parameter cannot be turned off #595

gelavat opened this issue Jun 29, 2024 · 5 comments
Assignees
Labels

Comments

@gelavat
Copy link

gelavat commented Jun 29, 2024

Is there an existing issue for this?

  • I have searched the existing issues

Describe the bug

Hello.
When I uncheck the "Activate statistics collection" switch, it is properly disabled but when I change the options tab and come back to the main parameters, it is unchecked again.

This is a bug, because all other parameters are properly kept, only this one is not being saved.

Can you please save this parameter too, each time it is switched ?

Regards.

Steps to reproduce

  1. Go to General parameters tab in the extension's options.
  2. Turn off the last parameter "activate statistics collection"
  3. Go to the Blacklist parameters tab
  4. Go back to the General parameters tab
  5. The "activate statistics collection" parameter is turned on again and in green. It should have been saved as turned off and displayed as turned off.

Browser

Firefox

Extension version

4.1.0

Anything else?

No response

@tarampampam
Copy link
Owner

Outch, the bug is confirmed. I'll make a fix ASAP

@gelavat
Copy link
Author

gelavat commented Jul 2, 2024

Thank you very much. And thanks for your great plugin!

@tarampampam
Copy link
Owner

@gelavat I've made a fix, but I'm still not completely sure it resolves the issue. Therefore, I'll keep this issue open even after the PR is merged and the release is published. Could you please confirm here if the issue is resolved after you receive the update? Thank you.

@tarampampam
Copy link
Owner

I have published a fresh release v4.1.1

@gelavat
Copy link
Author

gelavat commented Jul 6, 2024

Great, this fix works for me.
Thank you very much, all is fine now.

@gelavat gelavat closed this as completed Jul 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants