[17.0][MIG] stock_service_level

[damdam-s]

migrated from 14.0

[petrus-v]

Thanks for taking care of this migrations

[jbaudoux]

maybe just your name here or it should be Foodles?

Suggested change

	"author": "Pierre Verkest <[email protected]>,"
	"author": "Pierre Verkest,"

[petrus-v]

I'm fine with this suggestion

[jbaudoux]

Can you just keep copyright line? (same for other files)
You can put additional info in CREDITS file

[petrus-v]

We have already a lot of modules that use this pattern https://github.com/OCA/event/blob/15.0/event_sale_session/models/event_session.py#L3.

I'm fine if it's moved to a credit/author files and maybe with the initial author mentioned.

@jbaudoux is it an OCA convention? do you know why prefer that other ways ?

Also I've the feeling it's not the best moment to change that in a migration PR which add risk to get no response from the initial author and freeze migration task.

Finally we should probably add pre-commit hook to inform developers on that practices and eventually migrate existing case.

my 2cts

[jbaudoux]

As your are the author, it's fine for me if it's fine for you :)
I'm not aware of conventions

[jbaudoux]

/ocabot migration stock_service_level

[damdam-s]

@jbaudoux as part of migration, I'm not sure we should do it during the technical migration process.

@petrus-v any idea regarding this as you are the one being credited for this module 😄

[OCA-git-bot]

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