[14.0][IMP] purchase_tier_validation: Add decoration to purchase tree view when orders have been rejected or approved.

[victoralmau]

Add decoration to purchase tree view when orders have been rejected or approved.

Locked by:

• [14.0][IMP] base_tier_validation: Add validation_status field to tier.validation server-ux#730

Please @pedrobaeza and @chienandalu can you review it?

@Tecnativa TT33369

[pedrobaeza]

I don't think whole row should be decorated. This should be one optional column (hidden by default I would say) with such colors now that we have this option.

Waiting also for @LoisRForgeFlow's opinion on this.

[github-actions]

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

[LoisRForgeFlow]

I don't think whole row should be decorated. This should be one optional column (hidden by default I would say) with such colors now that we have this option.

Waiting also for @LoisRForgeFlow's opinion on this.

Sorry I missed this one. Yes, I like Pedro's option better, that new column can have colored bubbles depending on the validation status.

[victoralmau]

Sorry, I don't know exactly what you want to add, can you add a concrete example?

[LoisRForgeFlow]

@victoralmau I mean that you can do what Pedro suggested + take advantage of the decoration-success and decoration-danger in the field (to show a green or red background on the field)

Similar to most status fields in tree views (example in sales):

pseudocode: <field name="validation_status" decoration-danger=validatoin_status == "rejected" .../>

validation status doesn't exist, but can be added as a compute field in base_tier_validation if you think this approach is good.

[victoralmau]

Thanks, changes done.

[victoralmau]

Rebase done, ready for review.

[chienandalu]

👍

[pedrobaeza]

/ocabot merge minor

[OCA-git-bot]

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 14.0-ocabot-merge-pr-1440-by-pedrobaeza-bump-minor, awaiting test results.

[OCA-git-bot]

Congratulations, your PR was merged at 04eb26c. Thanks a lot for contributing to OCA. ❤️