Skip to content

Conversation

@nicolas-delbovier-acsone
Copy link

@nicolas-delbovier-acsone nicolas-delbovier-acsone commented Sep 8, 2025

This PR is based on #1597

  • New "Customer Quotation" message subtypes that we can subscribe partners to
  • Ensures salesman receives a notification for customer quotation actions
  • Returns a confirmation notification to a customer requesting a quotation
  • Notifies salesman when a user requests a quotation
  • Toggles between "CQ" workflow default message types and base default message types when toggling the use_customer_quotation_workflow variable.
image image

…otifications handeling for "CQ" workflow.

* New "Customer Quotation" message subtypes that we can subscribe partners to
* Ensures salesman receives a notification for customer quotation actions
* Returns a confirmation notification to  a customer requesting a quotation
* Notifies salesman when a user requests a quotation
* Toggles between "CQ" workflow default message types and base default message types when toggling the `use_customer_quotation_workflow` variable.
@nicolas-delbovier-acsone nicolas-delbovier-acsone changed the title [16.0][ADD] alc_sale_quotation_customer_quotation_workflow_notifications: notifications handeling for "CQ" workflow. [16.0][ADD] sale_quotation_customer_quotation_workflow_notifications: notifications handeling for "CQ" workflow. Sep 8, 2025
@sebastienbeau
Copy link
Contributor

Interesting feature, I will take the time to test it and review it. @nicolas-delbovier-acsone do you plan to be in OCA days next week ?

@nicolas-delbovier-acsone
Copy link
Author

Interesting feature, I will take the time to test it and review it. @nicolas-delbovier-acsone do you plan to be in OCA days next week ?

Yes I will be there, I would be happy to discuss this further in person then.

@sebastienbeau
Copy link
Contributor

Cool ! Let's discuss about this on Monday ;)

…ify code.

This commit removes a redundant check for `sale_order.use_customer_quotation_workflow` to simplify the code logic.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants