Skip to content

[19.0][FIX] sale_tier_validation: customers can sign approved quotations - #87

Open
bosd wants to merge 1 commit into
OCA:19.0from
bosd:19.0-fix-sale_tier_validation-portal-signature
Open

bosd wants to merge 1 commit into
OCA:19.0from
bosd:19.0-fix-sale_tier_validation-portal-signature

Conversation

@bosd

@bosd bosd commented Oct 6, 2026

Copy link
Copy Markdown
Contributor

A quotation stays under validation until it is confirmed, also once all its reviews are approved. When the customer signs it on the portal, Odoo writes signature, signed_by and signed_on, which tier validation refuses, so a quotation that needed a review could not be signed and accepted at all.

This allows those three fields under validation. Accepting still confirms the order, so a quotation whose reviews are still open still cannot be accepted. The test signs an approved quotation, and checks that other fields stay locked.

Found while writing the portal tests of #85; once this is merged, the exception those tests create for the signature fields can go.

A quotation stays under validation until it is confirmed, also once
all its reviews are approved. Signing it from the portal writes the
signature fields, which were refused, so the customer could not sign
and accept a quotation that went through tier validation.

Allow the signature fields under validation. Accepting still confirms
the order, so a quotation with open reviews still cannot be accepted.
@OCA-git-bot OCA-git-bot added series:19.0 mod:sale_tier_validation Module sale_tier_validation labels Oct 6, 2026
@bosd bosd added the bug Something isn't working label Oct 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working mod:sale_tier_validation Module sale_tier_validation series:19.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants