[17.0][ADD] sale_certification

[christian-ramos-tecnativa]

Module to handle the certification step in a contruction company
ROADMAP:

• Handle retentions
• Create Tests
• Improve README

[edescalona]

LGTM

[Yoana-RB]

Hello, could you please enable the Runboat? Thank you!

[christian-ramos-tecnativa]

Hi @edescalona can you rebase the branch and push it? Thanks!

[edescalona]

Hi @Yoana-RB @christian-ramos-tecnativa , runboat is now enabled.

[Yoana-RB]

Overall, the functionality is correct and the implementation is solid. The functional review is approved (LGTM). However, I have identified a few improvement points and pending aspects that would be worth reviewing, although these are non-blocking comments:

Translations: They do not appear to be implemented yet and should be added to ensure a consistent user experience across all supported languages.

UI improvements: The current certificate view could benefit from a clearer visual hierarchy. Adding a more representative title or header would improve readability and make it easier to identify the displayed information.
‎sale_certification/views/view_order_form_certify.xml

Grouped certifications: When certifying multiple orders, it may be worth considering support for grouped certificate management, either by allowing invoicing from a group of certificates or by grouping multiple orders within a single certificate. This would simplify scenarios where a consolidated invoice is generated from multiple orders.