Skip to content

Commit

Permalink
[IMP] purchase_tier_validation: Add validation_status field with badg…
Browse files Browse the repository at this point in the history 
…e widget to purchase tree view.

TT33369
  • Loading branch information
@victoralmau
victoralmau committed Oct 23, 2023
1 parent 43332ae commit f758498
Showing 1 changed file with 17 additions and 0 deletions.
17 changes: 17 additions & 0 deletions purchase_tier_validation/views/purchase_order_view.xml
View file
Original file line number Diff line number Diff line change
Expand Up @@ -24,4 +24,21 @@
</filter>
</field>
</record>
<record id="purchase_order_kpis_tree" model="ir.ui.view">
<field name="name">purchase.order.inherit.purchase.order.tree</field>
<field name="model">purchase.order</field>
<field name="inherit_id" ref="purchase.purchase_order_kpis_tree" />
<field name="arch" type="xml">
<tree position="inside">
<field
name="validation_status"
widget="badge"
optional="hide"
decoration-info="validation_status == 'pending'"
decoration-danger="validation_status == 'rejected'"
decoration-success="validation_status == 'validated'"
/>
</tree>
</field>
</record>
</odoo>

3 comments on commit f758498

@mrgoetz
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi Victor, we ran into problems with your commit using validation_status. Where is the attribute validation_status in purchase_order defined ? Do we miss a dependency ? We look forward hearing from you. Thanks.

@pedrobaeza
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Update base_tier_validation as well. Update should be always complete, not only certain modules.

@mrgoetz
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you Pedro and Victor. We had a script error. Everything is fine now.

Please sign in to comment.