Reset message state for Login and ResetPassword dialogs
close #1847
Where error messages for the Login dialog rendered in WelcomeScreen would persist after closing and re-opening the dialog. This is addressed by replicating the componentDidMount implementation already present in SignUp.component.js:
https://github.com/cboard-org/cboard/blob/2824ecf9f7749a5fe55a5af39dce66b2675663ac/src/components/Account/SignUp/SignUp.component.js#L40-L44
Hi @zkarmi! Thanks – this looks great. I've assigned @gonojuarez as the reviewer, so please wait for his feedback.
Just a quick note: for each PR, we usually reference the related issue using special keywords like closes, fixes, or resolves. The most commonly used one is closes, which will automatically close the issue when the PR is merged.
I gonna modify the description :)
Hi @zkarmi,
I'm currently migrating the following components from class-based to functional (object-based) components:
Login.component.jsResetPassword.component.jsSignIn.component.jsSignUp.component.js
Issue #1941
I saw your comment, and it might help me with an issue I'm facing in Login.component.js.
I’ll implement these changes in my Pull Request and will ask for your review. Thanks for helping us!
@zkarmi we are gonna implement this changes on the Gonza PRs. After that this PR will be closed
Sounds good @gonojuarez @RodriSanchez1!
This was solved on #1948 and #1951