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

[ADD][16.0] product_english_name module

Open bealdav opened this issue 1 year ago • 2 comments

screen_french

cc @legalsylvain @HaraldPanten @rousseldenis @Kev-Roche

bealdav avatar Sep 30 '24 14:09 bealdav

Thanks a lot, @bosd for your review

bealdav avatar Oct 03 '24 10:10 bealdav

image

Finally works

bealdav avatar Oct 03 '24 11:10 bealdav

Hi @bosd could you check again ?

bealdav avatar Oct 10 '24 13:10 bealdav

@bealdav You should squash your commits.

HaraldPanten avatar Oct 11 '24 13:10 HaraldPanten

@HaraldPanten thanks to notified me. Fixed

bealdav avatar Oct 11 '24 14:10 bealdav

Hi, the commit name should appear without the version number, something like:

[ADD] product_english_name module: description in few words

HaraldPanten avatar Oct 14 '24 07:10 HaraldPanten

@HaraldPanten where this rule is described ?

bealdav avatar Oct 14 '24 07:10 bealdav

https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#commit-message

pedrobaeza avatar Oct 14 '24 08:10 pedrobaeza

I don't see in the doc, that it's required to not add version for a new module.

But I'm ok to change my commit.

bealdav avatar Oct 14 '24 08:10 bealdav

I don't see in the doc, that it's required to not add version for a new module.

But I'm ok to change my commit.

Hi @bealdav We have to follow Odoo's tags, and then apply OCA rules in case there are no examples ([MIG] is one of these use cases).

You can see that Odoo never uses the version number in the commit messages, as well --> https://github.com/odoo/odoo/pulls and here you can see Odoo's guidelines (for tagging) --> https://www.odoo.com/documentation/16.0/contributing/development/git_guidelines.html#tag-and-module-name

HaraldPanten avatar Oct 14 '24 09:10 HaraldPanten

That's what I have applied, we're OK then [ADD] for adding new modules

bealdav avatar Oct 14 '24 09:10 bealdav

That seems to be nitpicking, but I'm OK to change my commit as said previously.

You can see that Odoo never uses the version number in the commit messages

bealdav avatar Oct 14 '24 09:10 bealdav

Done, thanks for reviews

bealdav avatar Oct 14 '24 12:10 bealdav

/ocabot merge nobump

HaraldPanten avatar Oct 14 '24 14:10 HaraldPanten

On my way to merge this fine PR! Prepared branch 16.0-ocabot-merge-pr-1732-by-HaraldPanten-bump-nobump, awaiting test results.

OCA-git-bot avatar Oct 14 '24 14:10 OCA-git-bot

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

OCA-git-bot avatar Oct 14 '24 15:10 OCA-git-bot