Skip to content
This repository has been archived by the owner on Nov 21, 2024. It is now read-only.

feat: use recaptcha v2 (invisible) #322

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

feat: use recaptcha v2 (invisible) #322

wants to merge 2 commits into from

Conversation

spaenleh
Copy link
Member

No description provided.

@spaenleh spaenleh requested a review from pyphilia March 12, 2024 15:51
Copy link

Quality Gate Passed Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

Copy link
Contributor

@pyphilia pyphilia left a comment

Choose a reason for hiding this comment

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

Thanks!! ⛏️

className="g-recaptcha"
data-sitekey={RECAPTCHA_SITE_KEY}
// eslint-disable-next-line no-console
data-callback={(token) => console.log('got token', token)}
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe use console.debug?

data-callback={(token) => console.log('got token', token)}
data-size="invisible"
>
hello
Copy link
Contributor

Choose a reason for hiding this comment

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

Have a better text here?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants