hypothesis / lms

LTI app for integrating with learning management systems
BSD 2-Clause "Simplified" License
46 stars 14 forks source link

Don't ask user to click "Authorize" on every page load #171

Closed seanh closed 5 years ago

seanh commented 6 years ago

The app is causing Canvas to show the user an OAuth 2 authorization dialog and ask the user to click Authorize whenever:

screenshot from 2018-08-13 17-32-22

It does not show the authorize dialog when a public URL (HTML or PDF) assignment is launched, though.

Repeatedly showing the authorize dialog is annoying for the user, reduces security, and creates a lot of access tokens in Canvas.

jeremydean commented 6 years ago

I'll just add that this is definitely something people notice and grumble about when using the app. I think it should be prioritized once we get to cleaning up app and not adding features. It'd greatly improve the crispness of the app from a UX POV.

jeremydean commented 5 years ago

Here's a user in the wild confused by this part of the workflow:

image

https://community.canvaslms.com/groups/designers/blog/2019/01/14/using-the-hypothesis-collaborative-annotation-tool-in-canvas

seanh commented 5 years ago

This is fixed now that the new_oauth feature flag has been released