search

Typehuman / SSEMMI

This is the public repository for the SSEMMI project: A decentralised and communitarian project aiming at pooling citizen science data in support of whale and environmental observations
MIT License
3 stars 9 forks source link

Login error handling #13

Open byrnenick opened 3 years ago

byrnenick commented 3 years ago

Log in response time is >15 seconds when testing.

This is particularly problematic when credentials are incorrect as it takes too long to find out that they were incorrect.

byrnenick commented 3 years ago

Could be due to work being performed on the database when testing; will retest once this has done and see if the performance issue remains.

aalaydrus commented 3 years ago

Had created a message pop-up just under the login input widget to signify if the website is contacting the database as an "attempt to log in". Will produce a message "Unable to login, please check your details or internet connection" if login fails and will produce a "Logging you in" if authentication succeeds.

byrnenick commented 3 years ago

We need to create better styling for this. Screenshot shared in Keybase for reference. Let's see if there's a banner that's available within the existing CSS framework that we are using.

What error cases are we considering, too?

Case 1: Usr/pw mismatch: "Your username or password did not match, please try again." Case 2: Server error/something else: "Something went wrong! Please try again shortly."

byrnenick commented 3 years ago

https://mdbootstrap.com/docs/vue/components/alerts/

aalaydrus commented 3 years ago

Have updated the login error message styling to utilise the styling of alerts from mdbootstrap.

Also, the messages and it's colour will display based on the aforementioned use case from @byrnenick's comment (user details error & network/something else.

NOTE on colours: