Skip to content

Conversation

@horgh
Copy link
Contributor

@horgh horgh commented Sep 16, 2025

No description provided.

horgh and others added 2 commits September 16, 2025 21:56
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>
@horgh horgh requested a review from Copilot September 16, 2025 22:07
Copy link

Copilot AI left a 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 Party enum to Event class with AGENT and CUSTOMER values
  • Added Method enum 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>
@horgh horgh force-pushed the wstorey/eng-2869-eventparty-and-paymentmethod-minfraud-inputs-are-supported branch from 415b745 to 253d5d1 Compare September 16, 2025 22:08
Copy link
Member

@oschwald oschwald left a 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>
@oschwald oschwald merged commit 427b9ad into main Sep 17, 2025
23 checks passed
@oschwald oschwald deleted the wstorey/eng-2869-eventparty-and-paymentmethod-minfraud-inputs-are-supported branch September 17, 2025 14:05
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