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

Updated Google Recaptcha #308

Open
chuycepeda opened this issue Dec 18, 2014 · 4 comments
Open

Updated Google Recaptcha #308

chuycepeda opened this issue Dec 18, 2014 · 4 comments

Comments

@chuycepeda
Copy link

Hi !

I've updated the google recaptcha method to the newest, coolest "I'm not a robot" version. Mainly I've changed captcha.py and also the captcha.displayhtml(..) calls.

I've never contributed to a repo like this so (1) perhaps you can help me to push it here or (2) you can pick code from https://github.com/chuycepeda/gae-boilerplate/blob/master/bp_includes/lib/captcha.py and look for captcha.displayhtml() calls to send only public_key argument at handlers.

Cheers,

Chuy

@nvictor
Copy link

nvictor commented Dec 29, 2014

thanks buddy,

maybe i was mistaken by the "actively developed". i have dealt with the new Google re-captcha and they are already pushing it. good to see someone that is keeping with all this craziness : )

@nvictor
Copy link

nvictor commented Dec 29, 2014

i have to take my good comment back for posterity : 3 sorry

really semi-colon and weird formatting in Python code ? : )

@chuycepeda
Copy link
Author

hahahaha can't tell why it works like charm on the appengine....

I went directly to check why there's the semi-colon and ¡surprise! it is on the production file which is working with no issues, interesting to notice...could the appengine override this typical issues?

I've already removed it, thanks for noticing it !

@coto
Copy link
Owner

coto commented Jan 8, 2015

Hi @chuycepeda Thanks... could you try to send a pull request with that please?

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

No branches or pull requests

3 participants