-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Fix: Suspending Account For TOS Violation Suspends Related PayPal Accounts For Fraud #2831
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
base: main
Are you sure you want to change the base?
Fix: Suspending Account For TOS Violation Suspends Related PayPal Accounts For Fraud #2831
Conversation
sm17p
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please have a look this is ready for review 🙏
| after_transition any => [:flagged_for_tos_violation], :do => :add_product_comment | ||
|
|
||
| after_transition any => %i[suspended_for_fraud suspended_for_tos_violation], :do => :suspend_sellers_other_accounts | ||
| after_transition any => :suspended_for_fraud, :do => :suspend_sellers_other_accounts |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Going by the spec, this seems safe unless I'm missing some key information.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
gumroad/app/sidekiq/check_payment_address_worker.rb
Lines 11 to 14 in a1c6014
| banned_accounts_with_same_payment_address = User.where( | |
| payment_address: user.payment_address, | |
| user_risk_state: ["suspended_for_tos_violation", "suspended_for_fraud"] | |
| ) |
Thanks for the PR. I think we need to update this as well, to make it consistent
I thought about that but I was not fully sure about it. My thought process was that if someone was suspended for TOS violation and they create/upate another account for selling, then they are clearly trying to commit a fraud? Maybe we can ask support before we route accounts suspended for TOS violation to flag for TOS, wdyt? |
Closes: #2830
Root Cause
We had a state transition setup for suspending related accounts for fraud whenever a creator was suspended for TOS violation
Solution
We now only suspend related accounts for fraud when a creator is suspended for fraud
Test Results
Checklist
AI Disclosure
No AI was used for this PR