Skip to content

VCST-3961: auto-generated transactionId for capture and refund documents#469

Merged
OlegoO merged 2 commits intodevfrom
feat/VCST-3961
Oct 8, 2025
Merged

VCST-3961: auto-generated transactionId for capture and refund documents#469
OlegoO merged 2 commits intodevfrom
feat/VCST-3961

Conversation

@OlegoO
Copy link
Contributor

@OlegoO OlegoO commented Oct 1, 2025

Description

feat: Added auto-generated transactionId for capture and refund documents.

References

QA-test:

Jira-link:

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

Artifact URL:

https://vc3prerelease.blob.core.windows.net/packages/VirtoCommerce.Orders_3.861.0-pr-469-ff91.zip

feat:  Added auto-generated transactionId for capture and refund documents.
@OlegoO OlegoO requested a review from Copilot October 1, 2025 12:59
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

Adds auto-generated sequential (zero-padded) transaction IDs for new capture and refund operations, replacing use of the base payment number.

  • Introduces logic to compute a suffixed transactionId based on payment.number and count of existing child operations.
  • Applies the generated transactionId to both the blade subtitle and the entity's transactionId for capture and refund blades.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
src/VirtoCommerce.OrdersModule.Web/Scripts/blades/refund-add.js Adds generation and assignment of a formatted transactionId for refunds
src/VirtoCommerce.OrdersModule.Web/Scripts/blades/capture-add.js Adds identical transactionId generation logic for captures

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.344
Timestamp: 01-10-2025T13:10:24

@sonarqubecloud
Copy link

sonarqubecloud bot commented Oct 8, 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: 7.478
Timestamp: 08-10-2025T12:34:18

@OlegoO OlegoO merged commit db85616 into dev Oct 8, 2025
7 checks passed
@OlegoO OlegoO deleted the feat/VCST-3961 branch October 8, 2025 12:59
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