Polkadex-Substrate / Polkadex-Open-Beta

12 stars 3 forks source link

Fix/disable password manager entry for trading password field #602

Open nashidahmed opened 10 months ago

nashidahmed commented 10 months ago

By raising this issue, I agree to the terms and conditions presented by Polkadex here: Terms and Conditions

Description

The trading password field does not correctly pass all 5 digits to the password manager. The app should allow password managers to save the trading password by passing all 5 digits. Currently by default, if I save the trading password on the password manager, it updates the password for the logged in email with the first digit of the trading password. Since I used a password manager to create my password for this email, I lost access to this email's password since it got replaced by the first digit of my trading password.

image

image

If the signup feature is going to be removed, this can be done later so that the issue of the email password being overwritten by the password manager doesn't exist. Then just allowing the password manager to save the trading password by passing all 5 digits should be good.

onchainaddress:{esoia6ocj8jC4Wq2mSeiCsXf1Z3eHqx7oHQ8hK8iPgHUYAmeC}