Skip to content

STF-1944: Add ryft payment processor - #510

Merged
mm-kevcenteno merged 1 commit into
mainfrom
greg/stf-1944-ryft-payment-processor
Nov 13, 2026
Merged

mm-kevcenteno merged 1 commit into
mainfrom
greg/stf-1944-ryft-payment-processor

Conversation

@oschwald

@oschwald oschwald commented Nov 13, 2026 •

Copy link
Copy Markdown
Member

Part of STF-1944.

Adds ryft (Ryft) to the minFraud /payment/processor values. The web service change is maxmind/mm_website#46792 on GitHub Enterprise.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • New Features
    • Payment processor validation now accepts ryft.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings November 13, 2026 16:21

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota. Try requesting a review at a lower effort level.

@coderabbitai

coderabbitai Bot commented Nov 13, 2026 •

Copy link
Copy Markdown

Explain Change →

No actionable comments were generated in the recent review.

Walkthrough

The _payment_processor allowlist now accepts ryft. The 3.4.0 changelog records this addition.

Changes: Payment processor validation

Layer / File(s) Summary
Add ryft to processor validation
src/minfraud/validation.py, HISTORY.rst
The _payment_processor allowlist accepts ryft. The 3.4.0 changelog records this value.

Estimated code review effort: 1 (Trivial) | ~4 minutes

Change: Feature

Merge Risk: Minimal · up to 6d8ac

This change lets the client accept ryft as a payment processor value and notes it in the changelog. No concrete risk was found in this repository. Release timing should still match the companion web service change.

Pre-merge checks: Passed 4

Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title "STF-1944: Add ryft payment processor" matches the main change. The PR adds "ryft" to the allowed _payment_processor values in src/minfraud/validation.py and records the change in HISTORY.rs…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Title check

Explanation

The title "STF-1944: Add ryft payment processor" matches the main change. The PR adds "ryft" to the allowed _payment_processor values in src/minfraud/validation.py and records the change in HISTORY.rst. The title names the feature and the ticket, and it is specific enough for a teammate scanning history.

Finishing Touches

Generate unit tests (beta)

  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

A rabbit checks the list today,
“Ryft” now joins the values in play.
One new entry, clear and neat,
The changelog marks the update complete.
Then off I hop on quickened feet.

Recent review info

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: fc140b0a-61f3-4c2d-bec4-f3d03b869289
Commits

Reviewing files that changed from the base of the PR and between 8ba535c and 6d8ac4c.

Files selected for processing: 2

  • HISTORY.rst
  • src/minfraud/validation.py

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

@mm-kevcenteno
mm-kevcenteno merged commit 16c7473 into main Nov 13, 2026
50 checks passed
@mm-kevcenteno
mm-kevcenteno deleted the greg/stf-1944-ryft-payment-processor branch November 13, 2026 17:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants