Skip to content
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

Adding the informative messages for the user #61

Closed
ymaheshwari1 opened this issue Aug 18, 2020 · 6 comments · Fixed by #95
Closed

Adding the informative messages for the user #61

ymaheshwari1 opened this issue Aug 18, 2020 · 6 comments · Fixed by #95
Assignees
Labels
CH20 Hakincodes easy

Comments

@ymaheshwari1
Copy link
Contributor

Describe the bug
When a user is logging out or logging in, there is no message displayed to the user whether the action is successfully performed or not.

To Reproduce
Steps to reproduce the behavior:

  1. Log In to your account
  2. You will be redirected to the Home Page but no message is displayed to the User about the activity.
  3. Log Out of your account.
  4. You will be redirected to the Home Page but no message is displayed to the User about the activity.

Expected behavior
A completion message needs to be displayed to the user. Like, You are logged In successfully or You are logged out successfully.

@ymaheshwari1
Copy link
Contributor Author

I am interested in working on this.

@harsh7in
Copy link
Owner

@ymaheshwari1 whare do you want to show the message?

@ymaheshwari1
Copy link
Contributor Author

@harsh-9in As the User is redirected to the Home screen, so we can show the message on the Home Screen

@Kajol-Kumari
Copy link
Collaborator

Kajol-Kumari commented Aug 19, 2020

@ymaheshwari1 how about showing this in the form of notification/toast message?

Reference: https://www.w3schools.com/howto/howto_js_alert.asp

@ymaheshwari1
Copy link
Contributor Author

@Kajol-Kumari Yes we can do that as well, we can use snackbar
https://www.w3schools.com/howto/howto_js_snackbar.asp

@Kajol-Kumari
Copy link
Collaborator

@ymaheshwari1 please go ahead with slackbar/notification/toast message :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CH20 Hakincodes easy
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants