Skip to content

Add Clear to the TransactionReportTag enum (STF-190)#390

Merged
horgh merged 1 commit intomainfrom
greg/stf-190
Apr 23, 2026
Merged

Add Clear to the TransactionReportTag enum (STF-190)#390
horgh merged 1 commit intomainfrom
greg/stf-190

Conversation

@oschwald
Copy link
Copy Markdown
Member

Summary

  • Adds Clear to the TransactionReportTag enum with its [EnumMember]
    mapping to "clear" and an XML doc summary.
  • Adds a unit test verifying the enum value is accepted on a
    TransactionReport.
  • Appends a bullet to the existing unreleased 6.0.0-beta1 entry in
    releasenotes.md.

The clear tag retracts a previously reported fraud report on a transaction
(restoring its label to "unknown", distinct from the positive NotFraud
signal). Backend support shipped in STF-15; this adds SDK support per STF-190.

Linear: https://linear.app/maxmind/issue/STF-190

Test plan

  • dotnet build MaxMind.MinFraud.sln — clean
  • dotnet run --project MaxMind.MinFraud.UnitTest ... -class "*TransactionReportTest" — 7 passed, 0 failed

🤖 Generated with Claude Code

Part of STF-190. The `clear` tag retracts a previously reported fraud
report tag on a transaction, restoring its label to "unknown" (distinct
from the positive `NotFraud` signal). Backend support shipped in
STF-15; this adds SDK support.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request adds a new Clear member to the TransactionReportTag enum, allowing users to clear previous transaction reports when initial classifications are incorrect. The update includes the necessary enum decoration for serialization, a unit test to verify the change, and an update to the release notes. I have no feedback to provide.

@horgh horgh merged commit 993f2a6 into main Apr 23, 2026
16 checks passed
@horgh horgh deleted the greg/stf-190 branch April 23, 2026 15:23
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.

2 participants