product-attribute icon indicating copy to clipboard operation
product-attribute copied to clipboard

[17.0][MIG] product_pricelist_fixed_currency_rate: Migration to 17.0

Open kobros-tech opened this issue 1 year ago • 3 comments

I had to troubleshoot many bugs and they are not from my side.

I installed pandoc manually. I modified a wrong code which is written in the documentation that describes how to use hooks.

kobros-tech avatar Aug 08 '24 22:08 kobros-tech

Hi @kobros-tech

Thanks for contributing. It looks good. One question though, any reason to covert all readme fragments to markdown?

@LoisRForgeFlow when I run precommit command it converts marked down file like contributers file to be marked down then it gets deleted when I commit.

I don't know why but this is what happens.

kobros-tech avatar Aug 09 '24 07:08 kobros-tech

@kobros-tech @LoisRForgeFlow @mymage @Ivorra78 @rvalyi

kobros-tech avatar Aug 13 '24 15:08 kobros-tech

Hi @kobros-tech Thanks for contributing. It looks good. One question though, any reason to covert all readme fragments to markdown?

@LoisRForgeFlow when I run precommit command it converts marked down file like contributers file to be marked down then it gets deleted when I commit.

I don't know why but this is what happens.

I didn't realize that this was automated until now... thanks!

LoisRForgeFlow avatar Aug 14 '24 06:08 LoisRForgeFlow

@mymage @LoisRForgeFlow @OCA-git-bot

I have added some tests for the new 17.0 version that were not covered in the previous 16.0 version. There was a red flag asked me to test some values and I made the update on the commit for this reason.

kobros-tech avatar Oct 13 '24 16:10 kobros-tech

@LoisRForgeFlow @mymage Can we merge the pull request?

/ocabot merge patch

kobros-tech avatar Nov 19 '24 06:11 kobros-tech

Sorry @kobros-tech you are not allowed to merge.

To do so you must either have push permissions on the repository, or be a declared maintainer of all modified addons.

If you wish to adopt an addon and become it's maintainer, open a pull request to add your GitHub login to the maintainers key of its manifest.

OCA-git-bot avatar Nov 19 '24 06:11 OCA-git-bot

/ocabot migration product_pricelist_fixed_currency_rate

rousseldenis avatar Nov 19 '24 07:11 rousseldenis

@kobros-tech For next time, it is better to put improvements in separate commits than migration one. That allows to easily backport or forwardport.

rousseldenis avatar Nov 19 '24 07:11 rousseldenis

/ocabot merge nobump

rousseldenis avatar Nov 19 '24 07:11 rousseldenis

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

OCA-git-bot avatar Nov 19 '24 07:11 OCA-git-bot

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

OCA-git-bot avatar Nov 19 '24 07:11 OCA-git-bot

@kobros-tech For next time, it is better to put improvements in separate commits than migration one. That allows to easily backport or forwardport.

Thanks for notification, I promise I will put this into my account.

kobros-tech avatar Nov 19 '24 10:11 kobros-tech