Add Forgot Password component #212
Conversation
|
Thanks for opening this Pull Request! A maintainer will review it shortly. Please make sure you've followed all contribution guidelines. Join our Discord Community for discussions, questions, and real-time collaboration! |
|
@jyothi-samberpu could you please explain what you have done how this feature will work also provide a video demo by simulating this feature |
|
Recording 2025-09-23 002053.mp4
<https://drive.google.com/file/d/1MCSpt6MdnB0m24qYNoTQ-xtAl_eXBc3T/view?usp=drivesdk>
Hi @SrigadaAkshayKumar, thank you for reviewing my PR.
I have added a ForgotPassword.jsx component that allows users to reset
their password using Firebase authentication.
…On Mon, 22 Sept 2025 at 23:54, Srigada Akshay Kumar < ***@***.***> wrote:
*SrigadaAkshayKumar* left a comment (SrigadaAkshayKumar/stock#212)
<#212 (comment)>
@jyothi-samberpu <https://github.com/jyothi-samberpu> could you please
explain what you have done how this feature will work also provide a video
demo by simulating this feature
—
Reply to this email directly, view it on GitHub
<#212 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BNQ5EBXJXY2CQIAKU7GA4GL3UA5G3AVCNFSM6AAAAACG74NSR2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTGMRQGY3TOMZWHE>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
|
Here’s a quick explanation of how the Forgot Password feature works: |
|
@SrigadaAkshayKumar This is the drive link Thoroughly go through the contribution I made and tell me if there is anything that I can change. |
|
@jyothi-samberpu actually it is fine but it as to show the message "Login successful" right then after it has to go to the home page |
|
@SrigadaAkshayKumar
https://youtu.be/z26Q4xjqEJ4?si=78IaMHEIiBHF8-sO
Take a look at this, and if everything is fine, I’ll send the request
again
…On Wed, 24 Sept 2025 at 19:40, Srigada Akshay Kumar < ***@***.***> wrote:
*SrigadaAkshayKumar* left a comment (SrigadaAkshayKumar/stock#212)
<#212 (comment)>
@jyothi-samberpu <https://github.com/jyothi-samberpu> actually it is fine
but it as to show the message "Login successful" right then after it has to
go to the home page
—
Reply to this email directly, view it on GitHub
<#212 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BNQ5EBTIO4M2ZL6QFNYXH6T3UKQTJAVCNFSM6AAAAACG74NSR2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTGMRYGY4DMNZWGI>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
2 participants
Pull Request for Stock Analyzer
Add Forgot Password functionality using Firebase authentication
Contributor Details
Name: Jyothi Samberpu
GitHub ID: jyothi-samberpu
LinkedIn Username : [jyothi-samberpu]
LinkedIn Profile URL: https://www.linkedin.com/in/jyothi-s-ab801124b/
Email ID (for communication): [email protected]
Describe Your Changes
Added ForgotPassword.jsx component for users to reset their password.
Updated firebase.js configuration for modular Firebase initialization.
Implemented mock Firebase handling to prevent app crashes if Firebase is not configured.
What Are The Files You Have Changed
src/components/ForgotPassword.jsx
src/components/firebase.js
src/App.js
package.json
package-lock.json
.env
How Has This Been Tested?
Tested by navigating to /forgot-password route and submitting a test email.
Verified that the reset email mock confirmation appears correctly.
App does not crash even if Firebase is not configured.
Type of Change
New feature (adds new functionality)
Bug fix (fixes Firebase config crash with mock
Checklist
My code follows the project’s guidelines and best practices.
I have performed a self-review of my code.
I have added meaningful comments where necessary.
I have updated documentation wherever applicable.
My changes do not generate any new warnings or errors.
I have included screenshots.