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

Added Toggle for Password Visibility and Implemented Functional Log Out Button #59

Merged
merged 4 commits into from
Oct 12, 2024

Conversation

abir499-ban
Copy link
Contributor

Issue Fix #54 : Display Log In / Log out button on Home page

This PR addresses the issue #54 which focusses mainly on adding a functional Log In button when the user is not logged In and similarly a Log out button when the user is Logged In. The following improvements have been made:

  1. Added a Log out button in the application which allows the logged in user to safely log out of the application
  2. Similarly, added a Log In button which directs the user to the /admin route to allow him/her to sign in, so that he/she does not have to manually go to the /admin route.
  3. Implemented the seamless feature of alternatively displaying the Log in or Log out button depending on the state of the user.
  4. Made sure that when the user is Logged in, he/she does not have access to the /admin route as the route allows the user to sign in into the application. A already signed in user having access to the Log in page via the /admin route is redundant.
  5. Implemented the Toggle button in the password input field, to switch between view and hide password while entering it.

Please consider this PR.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉 Boom! Your pull request just flew into the BlogLog HQ. High fives all around! Our team of tech wizards will check it out and get back to you faster than you can say 'code ninja!' Thanks for leveling up the project!

@jinx-vi-0
Copy link
Owner

@abir499-ban fix merge conflict

@abir499-ban
Copy link
Contributor Author

@jinx-vi-0 fixed merged conflicts 👍

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

Successfully merging this pull request may close these issues.

Display Log In/ Log Out Button on Home Page
2 participants