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

Feature request: option to exclude bots #21

Closed
Tracked by #11
MaluNoPeleke opened this issue Dec 20, 2021 · 9 comments
Closed
Tracked by #11

Feature request: option to exclude bots #21

MaluNoPeleke opened this issue Dec 20, 2021 · 9 comments
Labels
enhancement New feature or request

Comments

@MaluNoPeleke
Copy link

I really like this plugin but I am not so much interested in the bots data. Could you add an option in the panel or in the config to toggle bots data in the visualizations?

@Daandelange
Copy link
Owner

Thanks :)
Why not, sounds good, with a little work it could even be excluded from the database to keep that minimal too.

@Daandelange Daandelange added the enhancement New feature or request label Dec 20, 2021
@MaluNoPeleke
Copy link
Author

Looking forward to it, thanks for the quick reaction. I even found a very similar idea already mentioned in the options.php.

Daandelange added a commit that referenced this issue Jan 13, 2022
…isits, track referrers)

Option to disable tracking from localhost for #23 (still allowing to view the database)
Also start uniformising track() return bool.
Daandelange added a commit that referenced this issue Jan 13, 2022
@Daandelange
Copy link
Owner

There you go, there's now an option to hide the current bots data and to prevent collecting more.

@MaluNoPeleke
Copy link
Author

Thanks a lot. I would like to give it a try but composer is showing an error message while installing:

Class daandelange\SimpleStats\SimplestatsTrackingMode located in ./site/plugins/simplestats/src/models/SimpleStatsTrackingMode.php does not comply with psr-4 autoloading standard. Skipping.

Can you please fix that?

@Daandelange
Copy link
Owner

Oops, that should fix it, thanks.

@MaluNoPeleke
Copy link
Author

Thanks for the quick response. I need to wait until it is "visible" on packagist and will then update.

@MaluNoPeleke
Copy link
Author

Can you please push an update to packagist?

@Daandelange
Copy link
Owner

Done, but you'll have to use dev-master, no version bump yet.

@MaluNoPeleke
Copy link
Author

Great, works as desired, thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants