Skip to content

Regex fix for price filters#84

Merged
OlegoO merged 3 commits intodevfrom
fix/filter-syntax-mapper-regex
Jan 29, 2026
Merged

Regex fix for price filters#84
OlegoO merged 3 commits intodevfrom
fix/filter-syntax-mapper-regex

Conversation

@yuskithedeveloper
Copy link
Contributor

@yuskithedeveloper yuskithedeveloper commented Jan 21, 2026

Description

Changing the price filter regex so only price.<currencycode> gets rewritten, preventing accidental renames like PriceLevel.

References

QA-test:

Jira-link:

Artifact URL:

https://vc3prerelease.blob.core.windows.net/packages/VirtoCommerce.XCatalog_3.944.0-pr-84-12a6.zip

@vc-ci
Copy link
Contributor

vc-ci commented Jan 21, 2026

Review task created: https://virtocommerce.atlassian.net/browse/VCST-4548

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.731
Timestamp: 21-01-2026T16:38:58

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.548
Timestamp: 23-01-2026T12:08:40

@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.809
Timestamp: 26-01-2026T12:50:48

@OlegoO OlegoO merged commit 7c03529 into dev Jan 29, 2026
8 checks passed
@OlegoO OlegoO deleted the fix/filter-syntax-mapper-regex branch January 29, 2026 17:56
OlegoO pushed a commit that referenced this pull request Feb 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants