-
Notifications
You must be signed in to change notification settings - Fork 1
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
Center login page and show success and error message on login / forgot password #65
Conversation
@@ -133,6 +152,11 @@ const FormikSign = ({ classes, setAuthentication, setUser, props }) => ( | |||
resetForm(); | |||
} catch (error) { | |||
console.log(error); | |||
|
|||
enqueueSnackbar("Error on login, contact an admin", { |
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.
@sphinxs This message should me more clear to the user.
"contact an admin" sounds like an internal error, which is not true.
It's just the password (or e-mail) that are incorrect.
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.
Yep, I have removed the contact admin into another PR
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.
@iguit0 , I have also changed the logo from the application and fixed the bug when installing the dependencies (see package.json, the update for React Scrips), resolve #46 #49