-
Notifications
You must be signed in to change notification settings - Fork 20
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
Accept ToS during login flow v2 #989
Comments
So I assume the behavior of that url is what needs to be changed to include the terms of services modal. |
Looks like this is in |
My understanding is that normally terms of service are shown after the login. That does not seem easy in this case as after the login the server will already approve the desktop client. So we would need to hook into that somewhere to delay it until the TOS has been accepted. |
@nickvergessen what's your take on this? Any idea where we could hook into the server code? |
nextcloud/registration#293 could be used as inspiration. |
Instead it would be best to let user accept ToS before grant access
The text was updated successfully, but these errors were encountered: