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

Fix number can't be shared #93

Merged
merged 1 commit into from
Sep 23, 2024
Merged

Conversation

Mugisha146
Copy link
Contributor

@Mugisha146 Mugisha146 commented Sep 14, 2024

PR Description

This PR fix issue of multiple users from registering with the same phone number [https://github.com//pull/93].

Description of tasks that were expected to be completed

Bug Fix : To prevent multiple users from registering with the same phone number ( ensures each phone number can only be associated with a single user account).
This include add a validation checker that check if Telephone number already exists or used in register process by someone else then block he/him to register if number exist and display toast to notify him/her.

How has this been tested?

  1. Clone the repository.
  2. Checkout to the 117-fx-numbers-can-not-shared
  3. Run npm install, then npm run dev to verify the app is functioning correctly.
  4. Test the Register first and register at the second time with different details but same phone number then see if it work as expected.

With deployed Link: None

Track PR (issue number & link) : #117

-Issue #93

Screenshots (If appropriate especially on the frontend)
N/A

Copy link

@Ishimwe7 Ishimwe7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well done @Mugisha146 , it is now working

@ncutixavier ncutixavier merged commit 74f1bbc into develop Sep 23, 2024
1 check passed
@ncutixavier ncutixavier deleted the 117-fx-numbers-can-not-shared branch September 23, 2024 08:56
Salim-54 added a commit that referenced this pull request Sep 23, 2024
Salim-54 added a commit that referenced this pull request Sep 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants