Add captcha #1120
Add captcha #1120
Conversation
|
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/operation-code/operation-code/6l7bjwa18 |
📝Changed routes:
Commit 50dd095 (https://operation-code-git-fork-too-far-add-captcha.operation-code.now.sh). |
|
🤖 This is a bot 🤖 Built with commit 8926b49 https://deploy-preview-1120--operation-code-storybook.netlify.app |
There was a problem hiding this comment.
| @@ -60,6 +61,8 @@ class OperationCodeApp extends App { | |||
| componentDidMount() { | |||
| /* Analytics */ | |||
| // TODO: Leverage master-build-time-only env vars instead of window check | |||
| loadReCaptcha(); | |||
There was a problem hiding this comment.
Oh we won't wanna load this for every page. _app is like the middleware for every page's client run-time. It looks like we'll wanna call this function in the page that I asked for via:
useEffect(() => {
loadReCaptcha();
}, []);Check this out: https://reactjs.org/docs/hooks-effect.html
There was a problem hiding this comment.
Ok awesome! I remembered seeing that. I just couldn’t recall. Thanks!
Description of changes
This adds the recaptcha to the bottom of the Join page. Instantiated in _app.js, component created in ReCaptcha.js and placed on join.js. Followed the directions here.
Issue Resolved
Fixes #1117
Screenshots/GIFs