okpy / ok-client

A Python client for the OK autograding system

Home Page:https://okpy.org/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Require less user interaction upon first backup

jerryjrchen opened this issue · comments

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 :)

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

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