okpy
Require less user interaction upon first backup
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 [email protected]
Backup... 100% complete
Backup successful for user: [email protected]
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 :)
IIRC, the email prompt will auto select the Google account to use if they have multiple. It skips the Account Chooser OAuth page.
Hmm, I didn’t consider that. Okay, seems reasonable.
- Jerry
On May 3, 2018, at 11:45, Colin Schoen [email protected] wrote:
The email prompt will auto select the Google account to use if they have multiple.
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.
This is still a valid issue if you think it is better for them to just always be prompted with all available accounts.
Is there any way we can make it so you only have to do this process once per semester, instead of once per assignment? I think that would be the best solution.
- Jerry
On May 3, 2018, at 12:16, Colin Schoen [email protected] wrote:
This is still a valid issue if you think it is better for them to just always be prompted with all available accounts.
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/Cal-CS-61A-Staff/ok-client/issues/345#issuecomment-386406391, or mute the thread https://github.com/notifications/unsubscribe-auth/AIJr1DZOcJQqWyRxK7cN9hrhZVaI6B6_ks5tu1eCgaJpZM4TsxUH.