-
Notifications
You must be signed in to change notification settings - Fork 23
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
Add sample JSON file #79
Comments
It might be a good idea to have this implemented as a button (or other UI control) instead of credentials, to avoid a collision with a real username (particularly with #80 implemented) and so that it does not seem as though the credentials are being sent over to the Aspen instance. |
Perhaps this could be merged with #74, in that there could be a button on the login page to continue without sending credentials to Aspen, and we could then offer the options of continuing with the test account (a provided JSON export) or uploading one's own JSON export. |
A username collision cannot happen if the username is a format that is normally invalid for aspen, i.e. "bob". |
The problem with that is that we would be assuming that "bob" is invalid. ICTS could change the username format as they have done in the past. If a teacher's first initial is "B" and last name is "Ob", then "bob" would be a valid username under the current scheme (which may become important for us, if enough teachers are interested in having a teacher adaptation of Aspine). |
@notrodes This is not necessarily a backend issue, as nothing is being scraped from Aspen. |
This can be implemented as a suggested sample JSON file once #74 is merged. I have already begun working on such a file, but it is far from complete. |
Add test credentials that allow for one to explore the functionality of Aspine without exposing a real person's username and password / grades.
The text was updated successfully, but these errors were encountered: