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

Copy generated password to password confirmation field? #23

Open
scramatte opened this issue Jan 10, 2023 · 5 comments
Open

Copy generated password to password confirmation field? #23

scramatte opened this issue Jan 10, 2023 · 5 comments
Labels
bug Something isn't working improvement

Comments

@scramatte
Copy link

Hi,

Does it possible to copy generated password to password confirmation field?
May I can use my component for this ...
https://github.com/operativeit/nova-depend-fill

I'm going to try in parallel ...

Regards

@btrsco
Copy link
Contributor

btrsco commented Jan 12, 2023

Like the update in the other issue you created (#22). I will go through and test this out and try to determine if it's an issue on our end over the weekend or the other package you're using and potentially add support depending on the time investment. I'll let you know more later on, thanks!

@scramatte
Copy link
Author

Another option is to have password confirmation field inside the same component.

@btrsco
Copy link
Contributor

btrsco commented Jan 13, 2023

Good suggestion, I’ll see what I can come up with. Hopefully it’s just an easy fix vs an entirely new field.

@btrsco btrsco added bug Something isn't working improvement labels Jan 13, 2023
@scramatte
Copy link
Author

Moreover, when you toggle visibility, generate password, it will be easier to apply on both field

@btrsco
Copy link
Contributor

btrsco commented Feb 3, 2023

@scramatte, I can check compatibility for nova-depend-fill with the password generator but I would appreciate if you could provide an example of your implementation to ensure I'm testing with your code as closely as possible.

Thanks

@btrsco btrsco closed this as completed Feb 3, 2023
@btrsco btrsco reopened this Feb 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working improvement
Projects
None yet
Development

No branches or pull requests

2 participants