Skip to content

LG-9726 Use the fraud review checker in the place of fraud_review_eligible#8408

Merged
jmhooper merged 2 commits intomainfrom
jmhooper-replace-fraud-review-eligible
May 17, 2023
Merged

LG-9726 Use the fraud review checker in the place of fraud_review_eligible#8408
jmhooper merged 2 commits intomainfrom
jmhooper-replace-fraud-review-eligible

Conversation

@jmhooper
Copy link
Contributor

A previous commit (#8397) introduced the FraudReviewChecker which includes a method for checking that a user is eligible for fraud review. It did not end up getting used in the rake task where that check is performed. This commit does that and removes the old method from the user model.

jmhooper added 2 commits May 16, 2023 16:20
…gible

A previous commit (#8397) introduced the `FraudReviewChecker` which includes a method for checking that a user is eligible for fraud review. It did not end up getting used in the rake task where that check is performed. This commit does that and removes the old method from the user model.

[skip changelog]
Copy link
Contributor

@zachmargolis zachmargolis left a comment

Choose a reason for hiding this comment

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

LGTM

@jmhooper jmhooper merged commit 903f9ee into main May 17, 2023
@jmhooper jmhooper deleted the jmhooper-replace-fraud-review-eligible branch May 17, 2023 17:06
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

Successfully merging this pull request may close these issues.

2 participants