[12.0][ADD] cooperator_website_referral_source

OCA / cooperative

GNU Affero General Public License v3.0

11 stars 22 forks source link

[12.0][ADD] cooperator_website_referral_source #13

Closed victor-champonnois closed 2 years ago

victor-champonnois commented 2 years ago

Description

Add a "How did you hear about us?" question in the front-end form, the subscription request and the partner.

Odoo task (if applicable)

https://gestion.coopiteasy.be/web#id=9090&model=project.task&view_type=form&menu_id=

Checklist before approval

[ ] Tests are present (or not needed).
[ ] Credits/copyright have been changed correctly.
[ ] Change log snippet is present.
[x] (If a new module) Moving this to OCA has been considered.

victor-champonnois commented 2 years ago

Merci pour la review @remytms !

victor-champonnois commented 2 years ago

Tests are failing because cooperator_website is not merged yet

victor-champonnois commented 2 years ago

/ocabot rebase

OCA-git-bot commented 2 years ago

Sorry @victor-champonnois you are not allowed to rebase.

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.

remytms commented 2 years ago

/ocabot rebase

OCA-git-bot commented 2 years ago

@remytms The rebase process failed, because command git push --force coopiteasy tmp-pr-13:12.0-add-cooperator_website_referral_source failed with output:

remote: Permission to coopiteasy/cooperative.git denied to OCA-git-bot.
fatal: unable to access 'https://github.com/coopiteasy/cooperative/': The requested URL returned error: 403

remytms commented 2 years ago

@victor-champonnois I'm not sure why it try to access coopiteasy/cooperative repository, but it should be change somewhere. :)

victor-champonnois commented 2 years ago

@remytms can you approve the PR if it's OK for you ?

victor-champonnois commented 2 years ago

Tu peux la merger aussi (j'ai pas les droits :) )

remytms commented 2 years ago

/ocabot merge nobump

OCA-git-bot commented 2 years ago

This PR looks fantastic, let's merge it! Prepared branch 12.0-ocabot-merge-pr-13-by-remytms-bump-nobump, awaiting test results.

OCA-git-bot commented 2 years ago

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