[11.0][MIG] mail restrict follower selection

[phucngta]

No description provided.

[jcdrubay]

Should we switch to the shorter version?

# 2015 Therp BV <http://therp.nl>
# 2017 Komit <http://www.komit-consulting.com>
# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).`

Also, I am not sure if it is ok to add Komit to the copyrights for a migration.

@OCA/core-maintainers should there be any mention about the good practice in the migration guideline page: https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-11.0 ?

[phucngta]

In pull request OCA/web#738, @moylop260 asked me to keep original Copyright so i don't want to switch this to the shorter version.

[hbrunn]

it's fine to shorten this, keeping the copyright means not to delete the names. So I agree with @jcdrubay's proposal

[jcdrubay]

I think that Funders section can be removed as it is not applicable.

Komit is not a funder as their was no financial contribution, only time (and sweat).

[jcdrubay]

I don't think that Komit is not the author as Komit only takes care of the migration.

For a migration, I think that Komit should only be added as a contributor, in the README.rst

[hbrunn]

yes, author stays Therp BV and the OCA

[jcdrubay]

Please order the imports:

from odoo import api, models

[njeudy]

Code and test on my local database

[hbrunn]

please fix

module mail_restrict_follower_selection: creating or updating database tables
<string>:3: (WARNING/2) Explicit markup ends without a blank line; unexpected unindent.
2018-03-06 17:57:58,873 166 WARNING openerp_test odoo.addons.base.module.module: docutils' system message present: <system_message level="2" line="3" source="<string>" type="WARNING"><paragraph>Explicit markup ends without a blank line; unexpected unindent.</paragraph></system_message>

then we can merge

[phucngta]

@hbrunn It's fixed