-
Notifications
You must be signed in to change notification settings - Fork 1
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
Remove Anonymization Functionality from plugin #55
Comments
Removed Anonymization from admin in #62 As for having it in the plugin somewhere, it might be useful to keep it as a CLI command in case we want to run it locally, for example. TBD |
I agree that keeping this option in the CLI could be useful. What about adding a warning message prior to the execution of the anonymize() function to warn the user about the risks of this command and only proceed if they agree? Since these processes can take a bit of time, I think it could also be useful to provide some feedback in the terminal to show at least the action that is currently being performed (i.e: " |
I've submitted the PR #77 because I've found that the wp I've also:
We should be able to close this issue if the PR #77 gets merged. |
Anonymization has been removed from the plugin completely, so closing this for now. |
This is very cool functionality, but due to the limitations that we have from systems, we should probably remove this from the plugin. That way, we won't get someone trying to run it on a large site and getting a ping from systems again.
Also, removing references to it in the README.
The text was updated successfully, but these errors were encountered: