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
Redesign of all pages that are rendered by backend code #2319
Redesign of all pages that are rendered by backend code #2319
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
@@ -1,8 +1,16 @@ | |||
{% extends "layouts/signed_out.html" %} | |||
|
|||
{% block title %}Error :-({% endblock %} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we can remove this now. The alert icon serves a similar purpose.
redash/templates/invite.html
Outdated
<div class="fixed-width-page"> | ||
<div class="bg-white tiled"> | ||
<div class="m-b-default"> | ||
{% if google_auth_url %} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need to support SAML and the other providers here too (although SAML is the important one). How about, have two variations of the text:
Has SSO provider (Google, SAML, LDAP, etc) available: "To create your account, please choose a password or login with your SSO provider."
No SSO provider: "To create your account, please choose a password."
Then show the different login buttons as we do on the login page.
@arikfr Updated this PR. Hope I didn't miss anything when adding SSO login buttons to Invite user page but please review it. |
💃 |
…ages-redesign Redesign of all pages that are rendered by backend code
Login page:
Forgot password page:
Password reset page:
Invite user page:
Error page:
Setup page: