contract icon indicating copy to clipboard operation
contract copied to clipboard

[18.0][MIG] contract_price_revision

Open AlvaroRM11 opened this issue 11 months ago • 11 comments

@BinhexTeam Depends on: https://github.com/OCA/contract/pull/1136

AlvaroRM11 avatar Jan 20 '25 10:01 AlvaroRM11

Thanks for your contribution @AlvRM2002 , remember to check the migration guidelines and do it step by step. Missing the pre-commit stuff right now.

/ocabot migration contract_price_revision

rousseldenis avatar Jan 20 '25 10:01 rousseldenis

@AlvRM2002 I can do a code review after you get tests in green.

rrebollo avatar Jan 22 '25 15:01 rrebollo

@AlvRM2002 I can do a code review after you get tests in green.

What is required to get the tests green? It appears this PR is stuck waiting for the contract module to be updated to v18.

AdamBark avatar Feb 12 '25 15:02 AdamBark

@AlvRM2002 I can do a code review after you get tests in green.

What is required to get the tests green? It appears this PR is stuck waiting for the contract module to be updated to v18.

@AlvRM2002 @AdamBark In order to test this in runboat and make tests green, you need to follow this : https://github.com/OCA/maintainer-tools/wiki/Use-temporary-reference%28s%29-to-another-pull-request%28s%29

rousseldenis avatar Feb 12 '25 15:02 rousseldenis

@AlvRM2002 I can do a code review after you get tests in green.

What is required to get the tests green? It appears this PR is stuck waiting for the contract module to be updated to v18.

@AdamBark you can now try it with the runbot, thanks for your interest!

AlvaroRM11 avatar Feb 28 '25 14:02 AlvaroRM11

@AlvaroRM11 Thanks for this.

Could you put depending PR's link in this description in order to easy review? Thanks

rousseldenis avatar Mar 14 '25 12:03 rousseldenis

@AlvaroRM11 Thanks for this.

Could you put depending PR's link in this description in order to easy review? Thanks

Done!

AlvaroRM11 avatar Mar 14 '25 12:03 AlvaroRM11

@AlvaroRM11 To get green tests, you should include contract_line_successor PR in test-requirements.txt too

rousseldenis avatar Apr 24 '25 11:04 rousseldenis

can you rebase please

sbejaoui avatar Jun 04 '25 13:06 sbejaoui

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

OCA-git-bot avatar Jun 05 '25 22:06 OCA-git-bot

Can you review the CI and correct the errors in tests?

victoralmau avatar Jul 30 '25 15:07 victoralmau

Superseed by https://github.com/OCA/contract/pull/1290

victoralmau avatar Aug 25 '25 15:08 victoralmau