[ADD] New module: partner_is_company_auth_signup

[legalsylvain]

Add a new radiobox input in signup page, to know if the user is a company, or an individual. The feature is similar to the back-office main partner page.

It avoids to have bad information, when companies are creating accounts. (is_company is unchecked by default in Odoo Core)

[rvalyi]

interesting feature. Sure you prefer to propose it as AGPL rather than LGPL? AGPL is often a bit of a stretch for portal modules...

[legalsylvain]

interesting feature. Sure you prefer to propose it as AGPL rather than LGPL? AGPL is often a bit of a stretch for portal modules...

I don't think that a lot of modules will depends on this one. It is a quite trivial one.
And to a certain extent, I like the contamination of the AGPL. ;-)

[rvalyi]

interesting feature. Sure you prefer to propose it as AGPL rather than LGPL? AGPL is often a bit of a stretch for portal modules...

I don't think that a lot of modules will depends on this one. It is a quite trivial one.
And to a certain extent, I like the contamination of the AGPL. ;-)

well eventually l10n_br_portal may depend on it as this is company thing is something that used to cause troubles. But after we change l10n_br_base to LGPL we would like to keep l10n_br_portal as LGPL as well. So this could be a use case. But not a big deal either.

cc @marcelsavegnago

[legalsylvain]

well eventually l10n_br_portal may depend on it as this is company thing is something that used to cause troubles

Hi. I understand That is used to cause troubles. But I don't understand why you could add a dependency to that module.

This module just add a radio input on a form. no need to depend on it, just install it, IMO. Please, elaborate if you can.

thanks !

[rvalyi]

well eventually you licensed it as AGPL without thinking too much about it and was ready to change. That's why I asked. But sure enough if it stays AGPL and we aim to put l10n_br_portal as LGPL, then we wouldn't depend on it, no big deal as I said.

[maisim]

code review + live test from http://oca-partner-contact-18-0-pr2123-7c268faf2446.runboat.odoo-community.org

👏

[legalsylvain]

@hparfr, could you make a review and put a little 👍 on this one ? it is preventing the merge of #2118 that you reviewed.

thanks !

[legalsylvain]

/ocabot merge patch

[OCA-git-bot]

Sorry @legalsylvain 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.

[legalsylvain]

Hi some @OCA maintainers could merge this one ?
thanks !

[NL66278]

👍 LGTM

[NL66278]

/ocabot merge nobump

[OCA-git-bot]

This PR looks fantastic, let's merge it!
Prepared branch 18.0-ocabot-merge-pr-2123-by-NL66278-bump-nobump, awaiting test results.

[OCA-git-bot]

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