[17.0][ADD] sale_order_partner_invoice_period

[DantePereyra]

Adds a selector of period to invoice directions.

[dalonsod]

See comment, and also add pending requirements:

• For sales orders tree view, the field must be added, as optional and shown by default.
• For sales orders search view, we should be able to group by this new field.

[DantePereyra]

Changes done.

[dalonsod]

See comments

[DantePereyra]

Changes done.

[dalonsod]

👍 please add README stuff and I'll test it

[DantePereyra]

Done!

[dalonsod]

@DantePereyra during installation this error is raised:

odoo.tools.convert.ParseError: while parsing /home/david/Solvos/Desarrollo/CodeProjects/odoo-17-oca/slv-sale/sale_order_partner_invoice_period/views/sale_order_view.xml:16
Invalid Expression while parsing xpath "//search//filter[@name='order_month'"

View error context:
'-no context-'

Could you check?

[DantePereyra]

Done, it was an end bracket missing. It must work perfect now.

[dalonsod]

Make these four new tasks:

• Make invoice_period only available for sales orders tree view, not for quotations. If you inherit from sale.view_order_tree instead of sale.sale_order_tree (base view for both quotations and orders) you'll achieve the desired behavior.
• Place new field in tree view after invoice_status, that is a better location.
• Use widget="badge" for this field, as invoice_status does.
• Add default and es translations.

[DantePereyra]

All done.

[dalonsod]

Could you provide a screenshoot of some sales orders in the tree view with different invoice periods? I would like to see how badge widget is finally rendered

[DantePereyra]

There it is

[dalonsod]

OK... could you add some colors to badges, so each value has a different color? invoice_status field view definition has the key of that IMO. I have no preferences about every color...

[DantePereyra]

Changes done.

[dalonsod]

See minor comment, and I'll test it ASAP. Good work!

[DantePereyra]

Perfect, done and ready! Thanks!

[DantePereyra]

Changes done with new requirements.

[DantePereyra]

Added the last requirements.