Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Drop "HasTierPrices" product column #7187

Closed
AndreiMaz opened this issue May 13, 2024 · 1 comment
Closed

Drop "HasTierPrices" product column #7187

AndreiMaz opened this issue May 13, 2024 · 1 comment
Assignees
@exileDev
Labels
refactoring / source code
Milestone
Version 4.80

Comments

@AndreiMaz
Copy link
Member

AndreiMaz commented May 13, 2024 •
edited

There's no need to keep "HasTierPrices" column in the "Product" entity.

It was required for performance optimizarion when we used Entity Framework. Now we can cache values between HTTP requests. Hence we can remove this property. Just ensure that tier prices are properly cached between HTTP requests
@AndreiMaz AndreiMaz added this to the Version 4.80 milestone May 13, 2024
exileDev added a commit that referenced this issue Jun 11, 2024
@exileDev
#7187 Removed HasTierPrices column
f857182
@exileDev
Copy link
Contributor

Closed #7187

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

@exileDev exileDev

Labels
refactoring / source code
Projects
None yet
Milestone
Version 4.80
Development

No branches or pull requests
2 participants
@AndreiMaz @exileDev