[16.0][MIG] account_invoice_customer_no_autofollow

[sbidoul]

No description provided.

[sbidoul]

/ocabot migration account_invoice_customer_no_autofollow

[sbidoul]

We may want to rename this module to account_invoice_partner_no_autofollow since it equally applies to vendor bills.

[ivs-cetmix]

Code and functional LGTM. Pls check the comment regarding maintainers.

[ivs-cetmix]

We may want to rename this module to account_invoice_partner_no_autofollow since it equally applies to vendor bills.

@sbidoul this is a good idea. However I wouldn't do this to avoid potential migration issues.

[sbidoul]

I wonder why such module is not more popular. I kind of feel I'm missing something obvious...

@StefanRijnhart since you reviewed the same for sale orders, you may be interested in this one.

[StefanRijnhart]

Thanks! I see no issues with the rename, because there is no data associated with this module. It would be nice to see the config parameter renamed on migration (after using openupgrade's rename_modules) or (re)installation.

[ivs-cetmix]

Yup, this means that there we should add a migration script

Thanks! I see no issues with the rename, because there is no data associated with this module. It would be nice to see the config parameter renamed on migration (after using openupgrade's rename_modules) or (re)installation.

@StefanRijnhart Yup, actually we can keep the existing config parameter name or add a migration script it we really need to rename it.

[dessanhemrayev]

LGTM

[rafaelbn]

Hello @sbidoul

I wonder why such module is not more popular. I kind of feel I'm missing something obvious...

We don't have and request in clients. They have customers as followers and there are not any problem reported in many years.

🤔

Best regards!

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