Skip to content

VCST-4834: Use platform delete confirmation dialog for order entities#490

Open
basilkot wants to merge 1 commit intodevfrom
fix/VCST-4834
Open

VCST-4834: Use platform delete confirmation dialog for order entities#490
basilkot wants to merge 1 commit intodevfrom
fix/VCST-4834

Conversation

@basilkot
Copy link
Copy Markdown
Contributor

@basilkot basilkot commented Mar 26, 2026

Description

  • Migrated delete flows for orders list, operation detail, and address detail
  • Orders list uses data mode with count; operation and address details use name mode
  • Added order count localization key across all available languages

Test plan

  • Verify delete dialog for orders list shows order count
  • Verify delete dialog for operation detail shows operation type name
  • Verify delete dialog for address detail shows address name

References

QA-test:

Jira-link:

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

Artifact URL:

https://vc3prerelease.blob.core.windows.net/packages/VirtoCommerce.Orders_3.1003.0-pr-490-fbee.zip


Note

Low Risk
Low risk UI-only change that swaps the delete confirmation dialog implementation and adds localized display strings; main risk is incorrect dialog payload (name/count) causing missing text in the confirmation modal.

Overview
Switches order-related delete flows to use the platform showDeleteConfirmationDialog in the orders list, operation detail, and address detail blades.

The delete dialogs now pass richer context: the orders list supplies a data payload with the selected order count, while operation/address deletes pass a name for display. Adds a new orders.dialogs.orders-delete.order localization key across all supported languages to render the count string.

Written by Cursor Bugbot for commit fbeed29. This will update automatically on new commits. Configure here.

@sonarqubecloud
Copy link
Copy Markdown

Copy link
Copy Markdown
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.654
Timestamp: 26-03-2026T16:44:36

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.

2 participants