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

suggestion feature #29

Open
wants to merge 16 commits into
base: master
Choose a base branch
from
Open

suggestion feature #29

wants to merge 16 commits into from

Conversation

Kira-D
Copy link

@Kira-D Kira-D commented Aug 21, 2016

Don't forget to register production site at https://www.google.com/recaptcha/admin and make a wiki user page for this feature

@sushain97
Copy link
Member

The build is failing! :( Can you look into it, @Kira-D?

servlet.py Outdated
if recap == bypassToken:
logging.info('Adding data to wiki with bypass token')
else:

Copy link
Member

Choose a reason for hiding this comment

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

Extra line

@Kira-D
Copy link
Author

Kira-D commented Aug 23, 2016

@unhammer can you please help with an integration error. It seems something wrong with tornado library for python 3.2

@unhammer
Copy link
Member

merge (or cherry-pick) a4e8966

so use tornado 4.3 when python 3.2, otherwise newest tornado
@Kira-D
Copy link
Author

Kira-D commented Aug 24, 2016

Thanks!

Btw, I can see your fix and my code committed in master in one commit.
Is it ok?

@unhammer
Copy link
Member

unhammer commented Aug 24, 2016 via email

@jonorthwash
Copy link
Member

jonorthwash commented Aug 25, 2016

And from the point of view of GSoC, I think it doesn't really matter—in
case that was your concern.

On 24 August 2016 at 13:46, Kevin Brubeck Unhammer <notifications@github.com

wrote:

Btw, I can see your fix and my code committed in master in one commit.
Is it ok?

if I understand correctly, yeah – git prefers it being the same commit

@Androbin
Copy link
Contributor

Anyone still working on this?

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

Successfully merging this pull request may close these issues.

None yet

6 participants