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

Fix Issue 423: Otp Functionality enhancement with SQL Events #432

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

prajwal2431
Copy link
Contributor

📋 Description
Fix Issue #423

This PR introduces several key improvements, including the addition of an OTP generator module with automatic OTP generation, the implementation of OTP expiration functionality, and the integration of a timer in the UI to show remaining OTP validity. Additionally, the PR resolves an issue with the /resetpassword endpoint, renaming it to /reset. It also addresses a bug where repeated OTPs were sent in quick succession by implementing rate-limiting logic.

🔨 Changes Made
Added an OTP generator module for automatic OTP generation.
Implemented OTP expiration functionality to restrict OTP validity to a specific time period.
Integrated a timer in the UI to display the remaining time before OTP expiration.
Resolved the /resetpassword endpoint issue by renaming it to /reset to follow proper naming conventions.
Fixed database issues related to OTP generation and expiration, ensuring smooth functionality under extreme conditions.
Prevented repeated OTP sending by introducing rate-limiting logic.
✅ Checklist
Before submitting the PR, please make sure you have completed the following:

I have followed all the guidelines mentioned in CONTRIBUTING.md.
My code follows the style guidelines of this project.
I have performed a self-review of my own code.
I have commented my code, particularly in hard-to-understand areas.
I have made corresponding changes to the documentation (if applicable).
I have tested it locally and it works fine.
Any dependent changes have been merged and published in downstream modules.
🏷️ Types of Changes
What type of changes does your code introduce? (Check all that apply)

Bug fix (non-breaking change which fixes an issue) 🐛
[] New feature (non-breaking change which adds functionality) ✨
UI enhancement (non-breaking change which enhances UI) 🎨
Documentation update 📚
Fixes #423

@prajwal2431
Copy link
Contributor Author

@Harshdev098 Please review this PR, apologize for the previous issue, and update me on any changes.

@prajwal2431 prajwal2431 changed the title Fix Issue 143: Otp Functionality enhancement with SQL Events Fix Issue 423: Otp Functionality enhancement with SQL Events Jan 8, 2025
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.

Improving The OTP functionality
1 participant