appwrite / console

The Console that makes Appwrite tick from the browser 🖥
https://appwrite.io
BSD 3-Clause "New" or "Revised" License
315 stars 136 forks source link

🐛 Bug Report: Wrong Validation Message for API Key Field #973

Open ItzNotABug opened 4 months ago

ItzNotABug commented 4 months ago

👟 Reproduction steps

The Messaging Provider API Key field in the Settings indeed has a password type but the field itself isn't for Password. The validation text says - Password should contain at least 8 characters.

See -

messaging-api-key-field-error

👍 Expected behavior

The validation should say - API Key should contain at least 8 characters.

👎 Actual Behavior

The validation says - Password should contain at least 8 characters.

🎲 Appwrite version

Appwrite Cloud

💻 Operating system

Linux

🧱 Your Environment

No response

👀 Have you spent some time to check if this issue has been raised before?

🏢 Have you read the Code of Conduct?

naman1608 commented 1 month ago
image

The API Key field has a type of 'password' which is why this validation is shown. Should there even be a validation for this except when empty, because the API key will be copied from the source?

ItzNotABug commented 1 month ago

@naman1608, The issue explains that the field is indeed a password type, but the input validation shows "Password" while the field is called "API Key".

naman1608 commented 1 month ago

@naman1608, The issue explains that the field is indeed a password type, but the input validation shows "Password" while the field is called "API Key".

So we will have to change the type from 'password', but there is no other Input component right now for this use case ie: in which we can customize the validation messages. Maybe we can make a more generic Input component for such cases?