Skip to content

Conversation

@TusharKumawat1
Copy link

What does this PR do?

This pull request addresses a previously existing bug within the application, Where user can access the dashboard without logging to resolve this, I've taken a straightforward approach to address this issue. Our current implementation already includes a mechanism for determining the user's login status, which governs the toggling between the login and logout buttons. To enhance this behavior, I've introduced a useEffect within the relevant component. This useEffect now ensures that if a user is not logged in, they are automatically directed to the signup page.

Fixes #102

bug102.mp4

Requirement/Documentation

  • If there is a requirement document, please, share it here.
  • If there is a UI/UX design document, please, share it here.

Type of change

  • [x ] Bug fix (non-breaking change which fixes an issue)
  • Chore (refactoring code, technical debt, workflow improvements)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How should this be tested?

Mandatory Tasks

  • [ x] Make sure you have self-reviewed the code. A decent size PR without self-review might be rejected.

@github-actions github-actions bot added bug Something isn't working good first issue Good for newcomers hacktoberfest Hacktoberfest Accepted Issues labels Oct 21, 2023
@github-actions
Copy link

Thank you for following the naming conventions! 🙏

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

Labels

bug Something isn't working good first issue Good for newcomers hacktoberfest Hacktoberfest Accepted Issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Can access dashboard, even when not logged in

1 participant