Skip to content

Conversation

@pilarvargas-tecnativa
Copy link
Contributor

@pedrobaeza
Copy link
Member

/ocabot migration website_sale_financial_risk

@OCA-git-bot OCA-git-bot added this to the 18.0 milestone Dec 10, 2025
@OCA-git-bot OCA-git-bot mentioned this pull request Dec 10, 2025
12 tasks
Copy link

@eduezerouali-tecnativa eduezerouali-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 + tested on runboat. Just that minor thing.

@pilarvargas-tecnativa pilarvargas-tecnativa force-pushed the 18.0-mig-website_sale_financial_risk branch from 5a278ad to d706129 Compare January 15, 2026 15:17
Copy link
Member

@victoralmau victoralmau left a comment

Choose a reason for hiding this comment

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

Code and functional review OK.

It would be interesting to add a specific payment.method as demo data and link it to the corresponding payment.provider with the payment_method_ids field.

@pedrobaeza
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 18.0-ocabot-merge-pr-531-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit fb4d8de into OCA:18.0 Feb 3, 2026
5 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

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

@pedrobaeza pedrobaeza deleted the 18.0-mig-website_sale_financial_risk branch February 3, 2026 08:31
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.

6 participants