- 
                Notifications
    
You must be signed in to change notification settings  - Fork 8
 
Add /event/party, /payment/method, and new /event/type values #537
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
Add /event/party, /payment/method, and new /event/type values #537
Conversation
Added two new event types to support credit application submissions and fund transfer transactions in the minFraud API. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Added new party field to indicate whether an agent or customer is submitting the transaction. The party field accepts AGENT and CUSTOMER enum values and is serialized as lowercase in JSON requests. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
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.
Pull Request Overview
This PR adds new input fields and enumeration values to the MinFraud Java API, specifically adding /event/party and /payment/method fields along with new event type values.
- Added 
Partyenum to Event class with AGENT and CUSTOMER values - Added 
Methodenum to Payment class with various payment method types - Added CREDIT_APPLICATION and FUND_TRANSFER to the Event.Type enum
 
Reviewed Changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description | 
|---|---|
| src/main/java/com/maxmind/minfraud/request/Event.java | Added Party enum and party field with getter/setter methods | 
| src/main/java/com/maxmind/minfraud/request/Payment.java | Added Method enum and method field with getter/setter methods | 
| src/test/java/com/maxmind/minfraud/request/EventTest.java | Added tests for new party field and event type values | 
| src/test/java/com/maxmind/minfraud/request/PaymentTest.java | Added tests for new payment method field | 
| README.md | Updated example to show usage of new party field | 
| CHANGELOG.md | Documented the new features and breaking changes | 
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Added new method field to specify the payment method associated with the transaction. The method field accepts values like CARD, DIGITAL_WALLET, BUY_NOW_PAY_LATER, CRYPTO, and others, serialized as lowercase with underscores in JSON requests. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
415b745    to
    253d5d1      
    Compare
  
    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.
I think we should update src/test/resources/test-data/full-request*.json. I believe you did this for some of the other PRs, but I'll double check.
Edit: The others seem to have it, except Node, where I don' think we have an equivalent.
Updated JSON test files and RequestTestHelper to include the new party field (customer) in event objects and method field (card) in payment objects, ensuring complete test coverage. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
No description provided.