Skip to content

[16.0][IMP] sale_tier_validation: Hide Confirm button if the validation is pending or rejected#3619

Merged
OCA-git-bot merged 1 commit intoOCA:16.0from
Tecnativa:16.0-imp-sale_tier_validation-button-confirm
Jul 2, 2025
Merged

[16.0][IMP] sale_tier_validation: Hide Confirm button if the validation is pending or rejected#3619
OCA-git-bot merged 1 commit intoOCA:16.0from
Tecnativa:16.0-imp-sale_tier_validation-button-confirm

Conversation

@victoralmau
Copy link
Copy Markdown
Member

@victoralmau victoralmau commented Mar 11, 2025

Related to OCA/purchase-workflow#2605 (review)

Hide Confirm button if the validation is pending or rejected.

Before
antes

After
despues

Please @sergio-teruel can you review it?

@Tecnativa

@victoralmau victoralmau force-pushed the 16.0-imp-sale_tier_validation-button-confirm branch from 7a1c4e3 to 3e993f3 Compare March 19, 2025 07:34
Copy link
Copy Markdown
Contributor

@carlos-lopez-tecnativa carlos-lopez-tecnativa left a comment

Choose a reason for hiding this comment

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

LGTM

@victoralmau victoralmau force-pushed the 16.0-imp-sale_tier_validation-button-confirm branch from 3e993f3 to 2d65875 Compare July 1, 2025 14:10
@victoralmau
Copy link
Copy Markdown
Member Author

Finally this is ready to merge.

@OCA-git-bot
Copy link
Copy Markdown
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@pedrobaeza pedrobaeza added this to the 16.0 milestone Jul 2, 2025
@victoralmau victoralmau force-pushed the 16.0-imp-sale_tier_validation-button-confirm branch from 2d65875 to 417f4d1 Compare July 2, 2025 15:57
Copy link
Copy Markdown
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

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

/ocabot merge patch

@OCA-git-bot
Copy link
Copy Markdown
Contributor

On my way to merge this fine PR!
Prepared branch 16.0-ocabot-merge-pr-3619-by-pedrobaeza-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 2da7d37 into OCA:16.0 Jul 2, 2025
12 of 13 checks passed
@OCA-git-bot
Copy link
Copy Markdown
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants