8000 🐛 Bug Report: [Console/Auth] Unable to remove Phone Number or Email Address · Issue #6108 · appwrite/appwrite · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

🐛 Bug Report: [Console/Auth] Unable to remove Phone Number or Email Address #6108

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.

Sign up for GitHub

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

Jump to bottom
Closed
2 tasks done
ItzNotABug opened this issue Sep 1, 2023 · 3 comments
Closed
2 tasks done

🐛 Bug Report: [Console/Auth] Unable to remove Phone Number or Email Address #6108

ItzNotABug opened this issue Sep 1, 2023 · 3 comments
Assignees
@stnguyen90
Labels
bug Something isn't working product / auth Fixes and upgrades for the Appwrite Auth / Users / Teams services. product / console Console, UI and UX issues

Comments

@ItzNotABug
Copy link
Member
ItzNotABug commented Sep 1, 2023 •
edited
Loading

👟 Reproduction steps

  1. Go to any user that has both email address or a phone number, erase any one & click on Update.
    Errors:

    • Email: Invalid email: Value must be a valid email address.
    • Phone: Invalid number: Phone number must start with a '+' can have a maximum of fifteen digits.

  2. The main issue is that if you use an invalid phone number, it will be accepted.
    Example: Type in just country code like: +91, click update & it works!
    This isn't good if the account only has phone number as a user identifier.

👍 Expected behavior

  1. Allow removing of Phone numbers if there's an Email Address added to the account & vice-versa.
  2. Check the length of the phone number, the minimum acceptable length is around 7/8 digits [Source: Google Search].

👎 Actual Behavior

  1. Unable to remove any of the phone number or email address, if both exists!
  2. Phone numbers less than 7/8 digits are accepted.

🎲 Appwrite version

Version 1.3.x

💻 Operating system

Linux

🧱 Your Environment

Self hosted on local machine and a Ubuntu server.

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

  • I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?
@ItzNotABug ItzNotABug added the bug Something isn't working label Sep 1, 2023
@stnguyen90 stnguyen90 moved this to Todo in 1.4 release Sep 2, 2023
@stnguyen90 stnguyen90 modified the milestones: 1.4.2, 1.4.3 Sep 2, 2023
@joeyouss joeyouss added product / auth Fixes and upgrades for the Appwrite Auth / Users / Teams services. product / console Console, UI and UX issues labels Sep 3, 2023
@joeyouss
Copy link
joeyouss commented Sep 3, 2023 •
edited
Loading

Thanks for opening this @ItzNotABug
The main reason is the API https://appwrite.io/docs/server/users?sdk=nodejs-default#usersUpdatePhone that makes the params as required.

We have marked this for future releases.

@stnguyen90 stnguyen90 modified the milestones: 1.4.3, 1.4.4 Sep 12, 2023
@Megamannen Megamannen mentioned this issue Sep 13, 2023
Closed
2 tasks
@stnguyen90 stnguyen90 self-assigned this Sep 20, 2023
@github-actions github-actions bot mentioned this issue Oct 1, 2023
Closed
@techwithsrikanth
Copy link

Guess this is something related to schema of the table that is causing the issue, required field and maxLength/minLength can be updated in the fields to ensure this error can be resolved

Let me know your opinion and how's the issue currently going on?

@stnguyen90 stnguyen90 removed this from 1.4 release Mar 11, 2024
@stnguyen90 stnguyen90 moved this to Todo in 1.5 release Mar 11, 2024
@stnguyen90
Copy link
Contributor

Fixed as part of #7471

@stnguyen90 stnguyen90 mentioned this issue Mar 20, 2024
Merged
4 tasks
@github-project-automation github-project-automation bot moved this from Todo to Done in 1.5 release Mar 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@stnguyen90 stnguyen90

Labels
bug Something isn't working product / auth Fixes and upgrades for the Appwrite Auth / Users / Teams services. product / console Console, UI and UX issues
Projects
1.5 release
Status: Done
Milestone
No milestone
Development

No branches or pull requests
4 participants
@stnguyen90 @ItzNotABug @joeyouss @techwithsrikanth
0