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

Remove redundant captcha #1383

Merged
merged 3 commits into from Jun 24, 2019
Merged

Conversation

crhallberg
Copy link
Contributor

@crhallberg crhallberg commented Jun 19, 2019

I noticed that the Lightbox is adding an additional captcha to the form submissions. It's not harming anything but it is redundant.

@demiankatz
Copy link
Member

@crhallberg, did you look at the blame to try to figure out why this was there in the first place? Also note that we need to remove grecaptcha from the globals list if we don't use it anymore, or else Travis will get upset with us.

@demiankatz
Copy link
Member

Thanks, @crhallberg, I've also tested this, and everything seems fine, so I'm going to go ahead and merge.

@demiankatz demiankatz merged commit 9432525 into vufind-org:master Jun 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants