[16.0][ADD] cooperator_website_recaptcha#110
Conversation
Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>
Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>
Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>
Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>
* move the recaptcha widget view to portal_recaptcha. * remove res.company.captcha_type (portal_recaptcha can now be enabled and disabled in the settings). * override WebsiteSubscription._additional_validate() instead of .validation().
fix author and website manifest properties to follow oca's guidelines.
use website_recaptcha_v2 instead of portal_recaptcha.
|
/ocabot merge nobump |
|
On my way to merge this fine PR! |
|
@dreispt your merge command was aborted due to failed check(s), which you can inspect on this commit of 16.0-ocabot-merge-pr-110-by-dreispt-bump-nobump. After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red. |
|
@dreispt thanks for taking time for this. this pr is ready, but it depends on OCA/website#1032, which should be merged first. could you help with that? |
|
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
github-actions
Bot
added
the
stale
huguesdk
added
no stale
|
Hi all. This one can be merged. CC : @huguesdk |
|
/ocabot merge nobump |
|
On my way to merge this fine PR! |
|
@huguesdk your merge command was aborted due to failed check(s), which you can inspect on this commit of 16.0-ocabot-merge-pr-110-by-huguesdk-bump-nobump. After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red. |
|
@legalsylvain indeed, thanks for the heads up! it seems like there is a problem with the tests that prevents the merge. i’m going to look into it. |
|
/ocabot merge nobump |
|
On my way to merge this fine PR! |
|
Congratulations, your PR was merged at 6e2409b. Thanks a lot for contributing to OCA. ❤️ |
9 participants
move
cooperator_website_recaptchamodule from coopiteasy/vertical-cooperative to the oca (and migrate it to 16.0).this is the 16.0 version of #44.
tests will fail until OCA/website#1032 is merged.