Skip to content

VCST-3622: Reset password email should use correct store sender and template.#117

Merged
OlegoO merged 5 commits intodevfrom
fix/VCST-3622
Jul 31, 2025
Merged

VCST-3622: Reset password email should use correct store sender and template.#117
OlegoO merged 5 commits intodevfrom
fix/VCST-3622

Conversation

@OlegoO
Copy link
Contributor

@OlegoO OlegoO commented Jul 30, 2025

Description

fix: Reset password email should use the correct store sender and template by adding StoreId and LanguageCode to the password reset query, the frontend should pass them for correct notification template resolving.

query requestPasswordReset($storeId: String, $cultureName:String, $loginOrEmail: String!, $urlSuffix: String!) {
  requestPasswordReset(
    storeId: $storeId, 
    cultureName: $cultureName,
    loginOrEmail: $loginOrEmail, 
    urlSuffix: $urlSuffix)
}

{
  "loginOrEmail": "b2badmin@test.com",
  "urlSuffix": "/reset-password",
  "storeId": "B2B-store",
  "cultureName":"en-US"
}

References

QA-test:

Jira-link:

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

Artifact URL:

https://vc3prerelease.blob.core.windows.net/packages/VirtoCommerce.ProfileExperienceApiModule_3.919.0-pr-117-1329.zip

…emplate.

fix: Reset password email should use correct store sender and template by adding StoreId and LanguageCode to password reset query, the frontend should pass them for correct notification template resolving.
@OlegoO OlegoO requested a review from ksavosteev July 30, 2025 13:49
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.95
Timestamp: 30-07-2025T13:59:10

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.472
Timestamp: 30-07-2025T14:18:27

@sonarqubecloud
Copy link

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.516
Timestamp: 30-07-2025T15:31:12

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.627
Timestamp: 30-07-2025T15:41:11

@OlegoO OlegoO merged commit c8eb91d into dev Jul 31, 2025
8 checks passed
@OlegoO OlegoO deleted the fix/VCST-3622 branch July 31, 2025 14:49
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.

3 participants