feat: Add documentation for rule limit - #2540
Michael Telgmann (mitelg) wants to merge 1 commit into
Conversation
Developer Docs healthcheckStatus: Completed with |
8a6bfc0 to
59f5b15
Compare
Bojan Rajh (bojanrajh)
left a comment
There was a problem hiding this comment.
Michael Telgmann (@mitelg) Su (@sushmangupta) Micha Hobert (@Isengo1989) Since this is performance-oriented topic, should we move the article from Configurations to Performance section, at the same Guides > Hosting level?
|
Bojan Rajh (@bojanrajh) increasing the rule limit will decrease the performance. therefore I am not sure, it that would be suitable 😁 |
59f5b15 to
d87b52a
Compare
|
Copilot Make a decision about last 2 comments - flip a coin. :) |
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
Only a non-blocking documentation nit remains.
Review effort: Lite
Findings: None
What changed in this PR
Adds documentation for Shopware’s max_rule_prices setting, its default limit, configuration override, and performance impact.
Changes:
- Documents the default 100-rule limit.
- Adds YAML configuration guidance.
- Warns about potential listing performance effects.
| File | Description |
|---|---|
guides/hosting/configurations/shopware/price-rule-limit.md |
Documents the price-rule calculation limit and configuration example. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Tails — let’s keep it under |
Summary
Based on https://winkelwagen.de/2026/08/03/shopware-6-hidden-gems-7-shopware-dal-max_rule_prices-the-silent-price-sorting-cutoff/
Big kudos to Fabian Blechschmidt (@Schrank) 🚀 🎉
Checklist
PageRefreferences where relevant..gitbook.yamlif pages were moved, renamed, or deleted..wordlist.txt(and sorted it) if spellcheck flags new legitimate terms.