Skip to content

[17.0][MIG] website_recaptcha_v2 - #1102

Open
edescalona wants to merge 18 commits into
OCA:17.0from
BinhexTeam:17.0-mig-website_recaptcha_v2
Open

edescalona wants to merge 18 commits into
OCA:17.0from
BinhexTeam:17.0-mig-website_recaptcha_v2

Conversation

@edescalona

@edescalona edescalona commented Apr 21, 2025 •

Copy link
Copy Markdown

@BinhexTeam

Standard migration.

@rrebollo rrebollo left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review. LGTM!

@edescalona
edescalona force-pushed the 17.0-mig-website_recaptcha_v2 branch from c0b638d to d960b66 Compare May 20, 2025 20:27
@edescalona
edescalona requested a review from rrebollo June 16, 2025 19:32

@rrebollo rrebollo left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review. LGTM!

@antoniodavid antoniodavid left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@OCA-git-bot

Copy link
Copy Markdown
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

huguesdk and others added 18 commits December 10, 2025 08:26
rename website_recaptcha_reloaded to website_recaptcha_v2.
* add recaptcha widget view.
* allow to enable and disable recaptcha in settings.
* improve settings view.
* change arguments and return value of website.is_captcha_valid(): it
  now takes a form values dictionary as argument instead of a string and
  returns a tuple containing a boolean value and an error message
  instead of raising an exception.
* add tests.
* add recaptcha test keys as demo data.
* improve manifest and readme.
* use correct names for files, classes and xml ids.
Co-authored-by: hugues de keyzer <odoo@hugues.info>
set recaptcha script loading as async to prevent it from blocking the
page rendering.
Currently translated at 100.0% (19 of 19 strings)

Translation: website-16.0/website-16.0-website_recaptcha_v2
Translate-URL: https://translation.odoo-community.org/projects/website-16-0/website-16-0-website_recaptcha_v2/it/
@edescalona
edescalona force-pushed the 17.0-mig-website_recaptcha_v2 branch from c782213 to a888443 Compare December 10, 2025 13:27
@github-actions

Copy link
Copy Markdown

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 12, 2026
@github-actions github-actions Bot closed this May 17, 2026
@jelenapoblet jelenapoblet reopened this Oct 9, 2026
@OCA-git-bot OCA-git-bot added series:17.0 mod:website_recaptcha_v2 Module website_recaptcha_v2 labels Oct 9, 2026
@edescalona

Copy link
Copy Markdown
Author

Hi @jelenapoblet , could you try merging this PR?

@jelenapoblet

Copy link
Copy Markdown

Once we merge #1076 we can continue with this one.

@github-actions github-actions Bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Oct 11, 2026
@edescalona

Copy link
Copy Markdown
Author

Hi @jelenapoblet could you please try merge??

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants