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

Require less user interaction upon first backup #345

Open
jerryjrchen opened this issue Apr 30, 2018 · 4 comments
Open

Require less user interaction upon first backup #345

jerryjrchen opened this issue Apr 30, 2018 · 4 comments

Comments

@jerryjrchen
Copy link

The email prompt after running Ok seems unnecessary:

ERROR  | auth.py:91 | {'error': 'invalid_grant'}
Performing authentication
Please enter your bCourses email.
bCourses email: dslkfjak;lsfjiwaofnsf;jaksdfjaiofjaksdfnaefiiuajdisofjaKweofjsdofjdsio
Successfully logged in as jerry.c@berkeley.edu
Backup... 100% complete
Backup successful for user: jerry.c@berkeley.edu

After the first login (and I'm guessing as long as you save cookies), it seems like it doesn't really matter what the supplied bCourses email is. Could we just open the Ok webpage after a user is already set up?

Furthermore, perhaps there is a better way to get their email from the browser if it's first time setup.

This is a minor thing, but I think we could save quite a few keystrokes this way :)

@colinschoen
Copy link
Member

colinschoen commented May 3, 2018

IIRC, the email prompt will auto select the Google account to use if they have multiple. It skips the Account Chooser OAuth page.

@jerryjrchen
Copy link
Author

jerryjrchen commented May 3, 2018 via email

@colinschoen
Copy link
Member

This is still a valid issue if you think it is better for them to just always be prompted with all available accounts.

@jerryjrchen
Copy link
Author

jerryjrchen commented May 4, 2018 via email

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

2 participants