AnthonyNahas / ngx-auth-firebaseui

Angular Material UI component for firebase authentication
https://ngx-auth-firebaseui.firebaseapp.com
MIT License
589 stars 165 forks source link

Disable or hide 'Reset password' button on click #630

Closed Schwartzalexander closed 2 years ago

Schwartzalexander commented 3 years ago

Bug Report or Feature Request (mark with an x)

- [ ] bug report -> please search issues before submitting
- [X] feature request

OS and Version?

Windows 10

Versions

"ngx-auth-firebaseui": "5.3.0", Angular CLI: 12.1.3 Node: 14.15.3 Package Manager: npm 7.18.1 OS: win32 x64

Angular: 12.1.3 ... animations, cdk, cli, common, compiler, compiler-cli, core ... forms, localize, material, platform-browser ... platform-browser-dynamic, router, service-worker

Package Version

@angular-devkit/architect 0.1201.3 @angular-devkit/build-angular 12.1.3 @angular-devkit/core 12.1.3 @angular-devkit/schematics 12.1.3 @angular/fire 6.1.5 @angular/flex-layout 11.0.0-beta.33 @schematics/angular 12.1.3 rxjs 6.6.7 typescript 4.3.5

Repro steps

Without being logged in, click on "Forgot Password" in the component. Enter an appropriate email address. Click on "Reset Password" multiple times. Multiple password reset emails will be sent. Only the last one will contain a valid reset link.

The log given by the failure

No log.

Desired functionality

The button "Reset Password" should be disabled or hidden after the first click, so that no more than one email can be sent.

Blunderchips commented 3 years ago

The same applies to the login and registration buttons. On registration for example, if you click the button multiple times, the first click creates the account and the rest clicks cause errors as the account already exists.

AnthonyNahas commented 2 years ago

a PR is appreciated