Skip to content

VCST-3449: Add pickupLocationId to shipping#460

Merged
ksavosteev merged 8 commits intodevfrom
feat/VCST-3449
Jul 23, 2025
Merged

VCST-3449: Add pickupLocationId to shipping#460
ksavosteev merged 8 commits intodevfrom
feat/VCST-3449

Conversation

@basilkot
Copy link
Contributor

@basilkot basilkot commented Jul 1, 2025 •
edited by vc-ci
Loading

Description

Add PickupLocationid property to db entity and domain model

References

QA-test:

Jira-link:

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

Artifact URL:

https://vc3prerelease.blob.core.windows.net/packages/VirtoCommerce.Orders_3.853.0-pr-460-f33a.zip

@basilkot basilkot requested a review from artem-dudarev July 1, 2025 07:48
artem-dudarev
artem-dudarev requested changes Jul 1, 2025
View reviewed changes
artem-dudarev
artem-dudarev requested changes Jul 1, 2025
View reviewed changes
Copy link
Collaborator

@artem-dudarev artem-dudarev left a comment

Choose a reason for hiding this comment

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

Update module.manifest

vc-ci
vc-ci reviewed Jul 1, 2025
View reviewed changes
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.338
Timestamp: 01-07-2025T12:24:47

artem-dudarev
artem-dudarev previously approved these changes Jul 1, 2025
View reviewed changes
vc-ci
vc-ci reviewed Jul 7, 2025
View reviewed changes
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.446
Timestamp: 07-07-2025T11:57:26

vc-ci
vc-ci reviewed Jul 11, 2025
View reviewed changes
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.633
Timestamp: 11-07-2025T09:38:09

artem-dudarev
artem-dudarev previously approved these changes Jul 18, 2025
View reviewed changes
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jul 23, 2025

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
83.3% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

vc-ci
vc-ci reviewed Jul 23, 2025
View reviewed changes
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.52
Timestamp: 23-07-2025T15:33:15

@ksavosteev ksavosteev merged commit dde5e2b into dev Jul 23, 2025
7 checks passed
@ksavosteev ksavosteev deleted the feat/VCST-3449 branch July 23, 2025 15:43
vc-ci
vc-ci reviewed Jul 23, 2025
View reviewed changes
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.69
Timestamp: 23-07-2025T16:03:57

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vc-ci vc-ci vc-ci left review comments

@ksavosteev ksavosteev ksavosteev approved these changes

@artem-dudarev artem-dudarev artem-dudarev left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@basilkot @artem-dudarev @ksavosteev @vc-ci @Lenajava1