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

Webhook missing triggers #8116

Closed
SiR-DanieL opened this issue May 5, 2015 · 4 comments
Closed

Webhook missing triggers #8116

SiR-DanieL opened this issue May 5, 2015 · 4 comments
Assignees

Comments

@SiR-DanieL
Copy link
Member

When bulk updating orders (or changing the status via Actions buttons), or updating them via API, the webhook is not triggered.

Ticket #289909

cc @claudiosmweb

@claudiosanches
Copy link
Contributor

@maxrice is ok in adding these new triggers?

@maxrice
Copy link
Contributor

maxrice commented May 5, 2015

updating via the API should trigger the webhook, see https://github.com/woothemes/woocommerce/blob/master/includes/class-wc-webhook.php#L556-L564

Not sure about bulk updates in the admin or status changes, but yeah it would make sense to me to trigger updates when those actions occur 😸

@mikejolley
Copy link
Member

@justinshreve and number 2

@justinshreve
Copy link
Contributor

I have verified and tested that updating via the API does trigger the webhook.

Bulk status updates do not currently trigger the hook, but I have addressed this in #8303.

claudiosanches added a commit that referenced this issue Jun 5, 2015
Execute the 'order.updated' webhook when bulk editing and using the status action buttons (fixes #8116)
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

5 participants