Skip to content

Add GET and PATCH /tax-rules/{taxRuleId} API endpoints - #449

Open
luigimassa wants to merge 1 commit into
PrestaShop:devfrom
bwlab:feature/tax-rule-get-edit
Open

luigimassa wants to merge 1 commit into
PrestaShop:devfrom
bwlab:feature/tax-rule-get-edit

Conversation

@luigimassa

Copy link
Copy Markdown
Questions Answers
Description? Add GET and PATCH /tax-rules/{taxRuleId} API endpoints
Type? new feature
BC breaks? no
Deprecations? no
Fixed ticket? PrestaShop/PrestaShop#39630
Sponsor company @luigimassa bwlab.it
How to test? TaxRuleEndpointTest (CI)

Expose GetTaxRuleForEditing and EditTaxRuleCommand on the TaxRule resource. The read and
write formats match: stateIds is mapped on the single stateId of the query and command, and
the zip code range returned as two fields by the query is rebuilt as the "from-to" string the
commands accept.

Checked locally on PrestaShop 9.2.0: integration tests, PHPStan (phpstan-9.2.x.neon), PHP-CS-Fixer and Rector.

🤖 Generated with Claude Code

https://claude.ai/code/session_01MnP3JH7z8bCnTKA5veRzPX

Expose GetTaxRuleForEditing and EditTaxRuleCommand on the TaxRule resource. The read and
write formats match: stateIds is mapped on the single stateId of the query and command, and
the zip code range returned as two fields by the query is rebuilt as the "from-to" string the
commands accept.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@ps-jarvis

Copy link
Copy Markdown

Hello @luigimassa!

This is your first pull request on ps_apiresources repository of the PrestaShop project.

Thank you, and welcome to this Open Source community!

@ps-jarvis ps-jarvis added the Waiting for QA Status: Action required, Waiting for test feedback label Oct 29, 2026
@ps-jarvis ps-jarvis moved this from Ready for review to To be tested in PR Dashboard Oct 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Waiting for QA Status: Action required, Waiting for test feedback

Projects

Status: To be tested

Development

Successfully merging this pull request may close these issues.

3 participants