Skip to content

Update Invalid TransactionType Logic #46472

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

Merged
merged 1 commit into from
May 29, 2025
Merged

Conversation

AKapeniak
Copy link
Contributor

Example snippet returns an Invalid parse per row as follows:

"Deposit => Parsed Amount: 10000, New Balance: 10000
Invalid => Parsed Amount: 0, New Balance: 10000
Deposit => Parsed Amount: 500, New Balance: 10500
Invalid => Parsed Amount: 0, New Balance: 10500"

Summary

Added else statement and brackets around Invalid TransactionType logic

Example snippet returns an Invalid parse per row as follows:

"Deposit => Parsed Amount: 10000, New Balance: 10000
Invalid => Parsed Amount: 0, New Balance: 10000
Deposit => Parsed Amount: 500, New Balance: 10500
Invalid => Parsed Amount: 0, New Balance: 10500"
@AKapeniak AKapeniak requested review from BillWagner and a team as code owners May 29, 2025 04:45
@dotnetrepoman dotnetrepoman bot added this to the May 2025 milestone May 29, 2025
@dotnet-policy-service dotnet-policy-service bot added dotnet-csharp/svc get-started/subsvc community-contribution Indicates PR is created by someone from the .NET community. labels May 29, 2025
Copy link
Member

@BillWagner BillWagner left a comment

Choose a reason for hiding this comment

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

Thanks @AKapeniak

This LGTM and I'll :shipit: now.

@BillWagner BillWagner enabled auto-merge (squash) May 29, 2025 13:40
@BillWagner BillWagner merged commit 81e979c into dotnet:main May 29, 2025
12 checks passed
@AKapeniak AKapeniak deleted the patch-1 branch May 29, 2025 15:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community-contribution Indicates PR is created by someone from the .NET community. dotnet-csharp/svc get-started/subsvc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants