Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[12.0][ADD] cooperator_website_recaptcha #43

Open
wants to merge 12 commits into
base: 12.0
Choose a base branch
from

Conversation

huguesdk
Copy link
Member

@huguesdk huguesdk commented Dec 9, 2022

move cooperator_website_recaptcha module from coopiteasy/vertical-cooperative to the oca.

this branch contains an extra refactoring commit (from the 12.0-ref-cooperator_website_recaptcha_to_portal_recaptcha branch) that should be reviewed. this refactoring depends on #40 and OCA/website#914.

carmenbianca and others added 12 commits December 9, 2022 15:09
Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>
Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>
co-authored-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.
Copy link
Member

@carmenbianca carmenbianca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Side-note: There's no need to compile README.rst in your PRs. It is generated as a side effect of the ocabot merge command.

@github-actions
Copy link

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.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Apr 23, 2023
@github-actions github-actions bot closed this May 28, 2023
@huguesdk
Copy link
Member Author

@robinkeunen would you please re-open this pr?

@robinkeunen robinkeunen reopened this May 30, 2023
@github-actions github-actions bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Jun 4, 2023
@victor-champonnois
Copy link
Member

@huguesdk I wonder if we should spend time merging this refacto on 12 (plus the refacto of portal_recaptcha). We will merge it in 14 and then port it to 16. What do you think ?

@huguesdk
Copy link
Member Author

huguesdk commented Aug 7, 2023

@victor-champonnois as it is a module that is used in production by many clients and is one of the last ones that is still in coopiteasy/vertical-cooperative, i think it makes sense to complete this, to have the most used modules here in OCA/cooperative. moreover, it does not require much more work.

the only problem is that OCA/website#914, on which it depends, is not yet merged, and i don’t know if it ever will. let’s first work on getting the 14.0 version merged, then adapt the 12.0 version if it still makes sense at that time.

@robinkeunen: would you please apply the “no stale” label to this to ensure that it does not get automatically closed?

Copy link

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.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Dec 10, 2023
@huguesdk huguesdk added no stale Use this label to prevent the automated stale action from closing this PR/Issue. and removed stale PR/Issue without recent activity, it'll be soon closed automatically. labels Dec 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no stale Use this label to prevent the automated stale action from closing this PR/Issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants