Skip to content

Implemented a countdown timer with automatic redirection to the login…#121

Open
kamome-run wants to merge 2 commits intoalvarotorrestx:devfrom
kamome-run:feature/countdown-to-login
Open

Implemented a countdown timer with automatic redirection to the login…#121
kamome-run wants to merge 2 commits intoalvarotorrestx:devfrom
kamome-run:feature/countdown-to-login

Conversation

@kamome-run
Copy link

Implemented functions

  • Countdown timer after registration.
  • Transition to the login screen after the countdown timer reaches 0 seconds.
  • The countdown timer was set to 5 seconds.

Copy link
Owner

@alvarotorrestx alvarotorrestx left a comment

Choose a reason for hiding this comment

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

Vey nice implementation! There's just some minor changes and it's ready for merging! Thanks so much! 😀

Comment on lines +832 to +834
<div className="countdown-timer">
{isCountdownTimer && <CountdownTimer className="text-green-600 text-4xl" />}
</div>
Copy link
Owner

Choose a reason for hiding this comment

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

Let's have this replace lines 204 - 209 that you currently have.

Basically replacing: <p className="text-md mt-1"> <Link to="/login" className="link link-primary"> Click here to log in </Link>{" "} and get started! </p>

That way the new message is just your countdown and redirects inside of the same block as the success message.

Copy link
Author

Choose a reason for hiding this comment

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

The replacement is complete!

message: successMessage,
show: showSuccessToast,
showSuccess,
//showSuccess,
Copy link
Owner

Choose a reason for hiding this comment

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

Please uncomment this as we have had someone recently add a feature with the success on login page.

@kamome-run kamome-run requested a review from alvarotorrestx July 8, 2025 00:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants