Skip to content
This repository has been archived by the owner on Jul 12, 2024. It is now read-only.

Automated hook reference #2924

Closed
3 tasks
psealock opened this issue Sep 16, 2019 · 5 comments · Fixed by #6454
Closed
3 tasks

Automated hook reference #2924

psealock opened this issue Sep 16, 2019 · 5 comments · Fixed by #6454
Labels
category: extensibility type: documentation This issue is a request for better documentation.

Comments

@psealock
Copy link
Collaborator

psealock commented Sep 16, 2019

Create a script to scrape wc-admin codebase to compile hook docblocks and publish a reference.

  • PHP reference
  • Javascript reference
  • publish results

Background p90Yrv-17h

@psealock psealock added type: documentation This issue is a request for better documentation. category: extensibility [estimate] 8 and removed [estimate] 8 labels Sep 16, 2019
@webkul-amit
Copy link

Hello @psealock ,
in this update do the developers will be able to add custom filter like some sort of new dropdown using react for filtering data based on custom dropdown value and later adding filter to maupulate data at php end .

@webkul-amit
Copy link

cause what happening right now is i have to copy the whole code and then adding that filter to core. and modifying query myself according to need.
like i want a custom dropdown of vendor base reports so right now it isn't possible using some sort of custom filter.

@psealock
Copy link
Collaborator Author

@webkul-amit Thanks for writing. Reports do have a hook to add an Advanced Filter, similar to the one used in the Orders Report.

Screen Shot 2019-10-11 at 10 09 52 AM

You can use this to add or append query parameters. We are currently working on adding hooks to modify SQL statements themselves to modify report data. Would this be helpful? If not, can you go into more detail about what you'd like to see?

@github-actions
Copy link

This issue has been automatically marked as stale because it has not had any recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the status: stale The issue/PR is stale label Feb 27, 2021
@psealock psealock removed the status: stale The issue/PR is stale label Mar 2, 2021
@psealock
Copy link
Collaborator Author

psealock commented Mar 2, 2021

Not stale, thank you

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
category: extensibility type: documentation This issue is a request for better documentation.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants