Skip to content

pr-vcptcore-qa-02-04-2026-2fb1#5637

Open
vc-ci wants to merge 1 commit intovcptcore-qafrom
pr-vcptcore-qa-02-04-2026-2fb1
Open

pr-vcptcore-qa-02-04-2026-2fb1#5637
vc-ci wants to merge 1 commit intovcptcore-qafrom
pr-vcptcore-qa-02-04-2026-2fb1

Conversation

@vc-ci
Copy link
Copy Markdown
Contributor

@vc-ci vc-ci commented Apr 2, 2026

ci: Automated update vc-module-notification from PR undefined undefined


Note

Low Risk
Low risk: only updates a module artifact reference in backend/packages.json, with no code changes; main risk is deploying the wrong Notifications build if the version/tag is incorrect.

Overview
Updates backend/packages.json to switch the VirtoCommerce.Notifications module in the AzureBlob source from a PR-tagged artifact (3.1004.0-pr-194-8168) to the released 3.1004.0 zip, along with minor JSON formatting/ordering tweaks.

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

@CLAassistant
Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

},
{
"Id": "VirtoCommerce.Notifications",
"BlobName": "VirtoCommerce.Notifications_3.1004.0.zip"
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Notifications module blob name missing PR build suffix

High Severity

The BlobName for VirtoCommerce.Notifications changed from a PR prerelease build (VirtoCommerce.Notifications_3.1004.0-pr-194-8168.zip) to a plain release version (VirtoCommerce.Notifications_3.1004.0.zip). The PR description explicitly says "from PR undefined undefined," indicating the automation script failed to resolve the PR number and commit hash. Nearly all other AzureBlob modules use the -pr-{number}-{hash} suffix pattern. This likely points to a non-existent blob in the prerelease storage container, which would break deployment.

Fix in Cursor Fix in Web

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