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

added google login #5

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

added google login #5

wants to merge 1 commit into from

Conversation

Mila432
Copy link

@Mila432 Mila432 commented Jul 17, 2016

added google login + changed type used during rpc

@Mila432 Mila432 closed this Jul 17, 2016
@Mila432 Mila432 reopened this Jul 17, 2016
@ArchdukeTim
Copy link

I got an error about unknown4. Str has no attribute unknown4

@MonsterMMORPG
Copy link

hey @Mila432 ty very much for your efforts and helps on this project

my question is

How can i catch unhandled errors globally in Python?
Here the file i want to catch unhandled errors which terminates the application : http://pastebin.com/qvstX2Mz
So i can play a notification sound when an error occurred and my infinite search pokemon is discontinued

@Keda87
Copy link

Keda87 commented Jul 18, 2016

I'm trying copying your changes on this commit and I got an error IndexError: 'list index out of range' on this line.
fourth=r2.history[len(r2.history)-1].headers['Location'].replace('amp%3B','').replace('amp;','')
I have checked that r2 returns empty list when I'm trying login with google.

@Simon0x
Copy link

Simon0x commented Jul 18, 2016

nice help
but would use tejado#17, because its much cleaner
this is also merged into the base.
so maybe rebuild this branch with the base of tejado.

@marco-kretz
Copy link

Merge PR locally, worked fine, but I always get " problem in google login.." using my google user +pw :/

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.

6 participants