search

OCA / credit-control

Odoo modules for the customer credit control
GNU Affero General Public License v3.0
78 stars 199 forks source link

[17.0][MIG] sale_financial_risk #377

Closed sbiosca-s73 closed 5 months ago

sbiosca-s73 commented 6 months ago

Migrate sale_financial_risk module to version 17.0

pedrobaeza commented 6 months ago

https://github.com/OCA/maintainer-tools/wiki/Use-temporary-reference%28s%29-to-another-pull-request%28s%29

/ocabot migration sale_financial_risk

sbiosca-s73 commented 6 months ago

@carlosdauden changes done

sbiosca-s73 commented 6 months ago

@carlosdauden changes done

sbiosca-s73 commented 6 months ago

Not working test-requirements because the version of module account_financial_risk isn't correctly version

OCA-git-bot commented 5 months ago

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). 🤖

Danisan commented 5 months ago

@pedrobaeza good work!

pedrobaeza commented 5 months ago

You have to give thanks to the contributors doing the migration and the review.

@sbiosca-s73 please split the pre-commit changes from the migration itself as stated in the migration guide: https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-17.0

sbiosca-s73 commented 5 months ago

@pedrobaeza changes done. Thanks!

pedrobaeza commented 5 months ago

/ocabot merge nobump

OCA-git-bot commented 5 months ago

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

OCA-git-bot commented 5 months ago

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