You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Address the feedback in #6290 about providing a more specific error message that indicates the email address of the admin user we are using, when refreshing the admin token fails:
I think the error messages need to be more specific. The last thing we want is for the feature to stop working for the whole school and to brick-wall users with an overly-generic error message that doesn't give them the information they need to fix the problem.
See also this thread in Slack about issues seanh ran into while tesitng this.
The text was updated successfully, but these errors were encountered:
Just to add that I think the error message should also say what exactly the admin user needs to do to fix the problem. (I think they need to launch any Hypothesis Canvas Studio assignment.)
An addition to this. In early testing with a customer we ran into an issue where the admin had a mixed-case email address and the case of the email in the LTI launch didn't match that in the Canvas Studio settings. The initial launch of an assignment failed with a message that the admin had not authenticated. The problem would have been more obvious if the email address was present in the settings.
The code currently does a naive string comparison of email addresses, and is not aware of email providers which treat the prefix as case insensitive, or normalize prefixes in other ways for comparison (eg. ignoring periods as Gmail does).
Address the feedback in #6290 about providing a more specific error message that indicates the email address of the admin user we are using, when refreshing the admin token fails:
See also this thread in Slack about issues seanh ran into while tesitng this.
The text was updated successfully, but these errors were encountered: