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.

Sign up for GitHub

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

Jump to bottom

Add actions for Sift decisions #1

Merged
merged 27 commits into from
Nov 11, 2024
Merged

Add actions for Sift decisions #1

merged 27 commits into from
Nov 11, 2024

Conversation

chrisfromthelc
Copy link
Collaborator

Changes proposed in this Pull Request

  • This adds some actions to take on the user account and purchases depending on the Sift decision.

Testing instructions

  • TBA

Addresses: https://github.com/Automattic/gold/issues/503

@chrisfromthelc
Add utility functions for Sift decisions
6facfec
@chrisfromthelc
Update decision cases
0272d0e
@chrisfromthelc
Split file, create functions for each case
0f5a5a1
@chrisfromthelc
Fix linter errors
ceeebf4
@chrisfromthelc
More linter issues
ff52036
@chrisfromthelc
More linter issues v2
0571d04
@chrisfromthelc
More linter issues again
239509d
@chrisfromthelc
Hopefully fix linter issues
a9afb36
@chrisfromthelc
More linting
860f4c5
@chrisfromthelc
Ugh linting
949aafc
@chrisfromthelc
Word change
f4e0895
@chrisfromthelc
Fix linter issues
7de94ad
@chrisfromthelc
Merge branch 'trunk' into add/abuse-decision-actions
05345b9
@chrisfromthelc
Merge branch 'add/abuse-decision-actions' of github.com:a8cteam51/sif…
c305fb7 
…t-decisions into add/abuse-decision-actions
@chrisfromthelc
Fix bad merge
c0b9b90
@chrisfromthelc
😭
8da641a
@chrisfromthelc
Make decisions use actions instead of functions
306d00c
@chrisfromthelc
Remove .vscode/settings.json from version control
e2bab40
@chrisfromthelc
Rename file and actions
b06c061
@chrisfromthelc
Adjust namespaces
785e876
@chrisfromthelc
Move files to separate folder
2496b05
@chrisfromthelc
Fix comment
4b84d83
@chrisfromthelc
Naming changes
57ccb45
@chrisfromthelc chrisfromthelc self-assigned this Nov 11, 2024
@chrisfromthelc chrisfromthelc mentioned this pull request Nov 11, 2024
Closed
@millerf
Copy link
Collaborator

millerf commented Nov 11, 2024

Needs some rebase (not sure why it doesn't show ⬆️ ....

@github-actions GitHub Actions
Copy link

Code Coverage

Package Line Rate Complexity Health
inc/abuse-decisions-actions.php 0% 22 ❌
inc/abuse-decisions.php 0% 12 ❌
inc/rest-api-webhooks.php 0% 5 ❌
inc/sift-object-validator.php 92% 123 ✔
inc/tracking-js.php 0% 4 ❌
inc/wc-settings-tab.php 0% 14 ❌
inc/woocommerce-actions.php 71% 85 ➖
index.php 0% 0 ❌
sift-for-woocommerce.php 0% 9 ❌
Summary 61% (676 / 1108) 274 ➖

millerf
millerf approved these changes Nov 11, 2024
View reviewed changes
Copy link
Collaborator

@millerf millerf left a comment

Choose a reason for hiding this comment

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

LGTM!

@chrisfromthelc chrisfromthelc merged commit 1936568 into trunk Nov 11, 2024
12 checks passed
@chrisfromthelc chrisfromthelc deleted the add/abuse-decision-actions branch November 11, 2024 17:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@millerf millerf millerf approved these changes

Assignees

@chrisfromthelc chrisfromthelc

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@chrisfromthelc @millerf