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

Add $order_status event #10

Merged
merged 3 commits into from
Oct 31, 2024
Merged

Add $order_status event #10

merged 3 commits into from
Oct 31, 2024

Conversation

n3f
Copy link
Collaborator

@n3f n3f commented Oct 30, 2024

Changes proposed in this Pull Request

  • Add $order_status validation.

Testing instructions

  • PHPUnit tests included.

Mentions https://github.com/Automattic/gold/issues/502

Base automatically changed from events/validate to trunk October 30, 2024 16:48
@n3f n3f self-assigned this Oct 30, 2024
Copy link

Code Coverage

Package Line Rate Complexity Health
inc/abuse-decisions-actions.php 0% 22
inc/abuse-decisions.php 0% 19
inc/rest-api-webhooks.php 0% 5
inc/sift-object-validator.php 92% 122
inc/tracking-js.php 0% 4
inc/wc-settings-tab.php 0% 14
inc/woocommerce-actions.php 71% 87
index.php 0% 0
sift-decisions.php 0% 9
Summary 60% (677 / 1129) 282

@n3f n3f marked this pull request as ready for review October 30, 2024 23:20
Copy link
Collaborator

@chrismccluskey chrismccluskey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@n3f n3f merged commit 6d9bd47 into trunk Oct 31, 2024
12 checks passed
@n3f n3f deleted the event/order-status branch October 31, 2024 21:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants