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

Just a FYI #1

Closed
w3guy opened this issue Aug 22, 2016 · 2 comments
Closed

Just a FYI #1

w3guy opened this issue Aug 22, 2016 · 2 comments

Comments

@w3guy
Copy link

w3guy commented Aug 22, 2016

@scottopolis (This](https://github.com/scottopolis/edd-metrics/blob/master/edd-metrics.php#L230) isn't 100 percent true as you can replace 'EDD_Metrics_activation' function with an array with the first value set to the class instance and the second the class method to call.

Read my post on WordPress hook system https://www.sitepoint.com/understanding-namespaces-wordpress-hook-system/

Btw nice work with this plugin. Love the graph and i just might copy the design of how stats are display in my next project 😄

@scottopolis
Copy link
Owner

Hmm, this is boilerplate code from Pippin, I'll have to ping him on this one.

Thanks!

@w3guy
Copy link
Author

w3guy commented Aug 24, 2016

Just thought I should let you know. We all learn everyday.

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

No branches or pull requests

2 participants