-
Notifications
You must be signed in to change notification settings - Fork 131
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
🐛 Bug Report: Wrong Message When Changing Email on Console #975
Comments
Hi there, I would like to work on this issue, please. |
Nevermind this bug seems to have been fixed already - should be marked as complete / solved. |
@ItzNotABug, thanks for creating this issue! 🙏🏼 You mentioned:
We actually don't want to say the email address already exists because that let's the user know someone has that email on the platform. As such, can this be closed? |
@stnguyen90, We could instead use something or similar like -
|
@gewenyu99, what are your thoughts on the copy of the error message? |
@stnguyen90 It's interesting because the solution LinkedIn, Facebook, etc use is actually just a captcha that blocks repeated attempts to snoop for emails. I really want to be able to display a different response to the Console SDK vs Client/Server SDK. For for, |
@gewenyu99, to do this, the Console needs to check the type (assuming the type is unique enough) and then show some custom message, ignoring the message from the API response. |
@stnguyen90 Does this API exist for client? Doesn't it require a password? Maybe we just need to rate limit it? |
@gewenyu99, yes, the update email endpoint requires a password. There must be a default rate limit on it. |
👟 Reproduction steps
Assuming you have 2 different Email Addresses, each with an Account on Cloud.
There was an error processing your request. Please check the inputs and try again.
The error should properly depict that the new Email Address already exists on Appwrite or connected to an Organization.
👍 Expected behavior
Show proper error message as mentioned above.
👎 Actual Behavior
Current shown error is not too clear regarding the actual error.
🎲 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?
The text was updated successfully, but these errors were encountered: