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

Added options to search WooCommerce log files if WC was detected #295

Open
wants to merge 1 commit into
base: trunk
Choose a base branch
from

Conversation

primathon
Copy link
Contributor

I often find myself needing to search through WC log files in order to find specific strings (order IDs, customer email addresses, etc). This PR adds an option to search through the /wp-content/uploads/wc-logs directory if the wc_get_logger() function is detected.

@Clorith
Copy link
Contributor

Clorith commented Aug 7, 2022

Hiya, and thanks for the Pull Request!

This is a very interesting concept, so far, the plugin hasn't catered specifically to any one plugin like this.

I wonder, are there other major plugins that do similar things that it would make sense to add custom handling for along the same lines?

@primathon
Copy link
Contributor Author

None that I can think of off the top of my head, but given how popular WooCommerce is (basically the de facto standard for ecommerce in WordPress), I think it makes sense to include something like this.

The only other feature I find myself really missing in this plugin is the ability to limit the search based on filetype, but it looks like there's still an open issue referencing this from 6 months ago here.

@Clorith
Copy link
Contributor

Clorith commented Aug 21, 2022

I've been thinking a fair bit on this, and opened #310 to do some further discussing (since a PR should primarily be about the pull request it self, and not discussion about implementation in my mind :) )

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