Skip to content

VCST-3999: Improve refund data model#467

Merged
OlegoO merged 3 commits intodevfrom
feat/VCST-3999
Oct 8, 2025
Merged

VCST-3999: Improve refund data model#467
OlegoO merged 3 commits intodevfrom
feat/VCST-3999

Conversation

@OlegoO
Copy link
Contributor

@OlegoO OlegoO commented Sep 24, 2025

Description

feat: Added TransactionId into CapturePaymentRequest and RefundPaymentRequest Parameters to pass gateway-specific metadata to the payment provider.

References

QA-test:

Jira-link:

https://virtocommerce.atlassian.net/browse/VCST-3999

Artifact URL:

https://vc3prerelease.blob.core.windows.net/packages/VirtoCommerce.Orders_3.860.0-pr-467-2f91.zip

feat: Added TransactionId  into CapturePaymentRequest and RefundPaymentRequest Parameters to pass gateway-specific metadata to the payment provider.
@cursor
Copy link

cursor bot commented Sep 24, 2025

You have run out of free Bugbot PR reviews for this billing cycle. This will reset on October 11.

To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

@OlegoO OlegoO requested a review from Copilot September 24, 2025 10:30
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 improves the refund data model by adding TransactionId support to payment processing methods and optimizing parameter handling.

  • Adds TransactionId to both CapturePaymentRequest and RefundPaymentRequest parameters
  • Refactors parameter handling to use more efficient dictionary initialization syntax
  • Removes unused System.Collections.Specialized import

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Contributor

@vc-ci vc-ci left a comment

Choose a reason for hiding this comment

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

Test Suite: Test Suites/Modules/module_Assets
Tests: 13
Failures: 0
Errors: 0
Time: 8.32
Timestamp: 24-09-2025T10:34:59

Copy link
Contributor

@vc-ci vc-ci left a comment

Choose a reason for hiding this comment

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

Test Suite: Test Suites/Modules/module_Assets
Tests: 13
Failures: 0
Errors: 0
Time: 7.103
Timestamp: 24-09-2025T10:55:22

@sonarqubecloud
Copy link

sonarqubecloud bot commented Oct 7, 2025

Copy link
Contributor

@vc-ci vc-ci left a comment

Choose a reason for hiding this comment

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

Test Suite: Test Suites/Modules/module_Assets
Tests: 13
Failures: 0
Errors: 0
Time: 8.071
Timestamp: 07-10-2025T15:45:29

@OlegoO OlegoO merged commit 26d0ca7 into dev Oct 8, 2025
7 checks passed
@OlegoO OlegoO deleted the feat/VCST-3999 branch October 8, 2025 08:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants