Skip to content

  • Projects
  • Groups
  • Snippets
  • Help
  • Sign in / Register
Minds Backend - Engine
Minds Backend - Engine
  • Project overview
  • Repository
  • Issues 275
  • Merge Requests 33
  • CI / CD
  • Security & Compliance
  • Packages
  • Analytics
  • Wiki
  • Snippets
  • Members
  • Collapse sidebar
  • Graph
  • Charts
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • Minds
  • Minds Backend - EngineMinds Backend - Engine
  • Merge Requests
  • !470

Open
Opened 1 hour ago by Marcelo Rivera@eiennohi
Report abuse

Change the error message for "No VOIP allowed" to "Invalid phone number"

  • Overview 0
  • Commits 1
  • Pipelines 1
  • Changes 4

closes #1319

** Testing ** We need to test verification for:

  • twofactor
  • rewards (in Wallet)
  • Onboarding

So do this for twofactor in settings, rewards and onboarding:

  1. Create a new user
  2. Go to wallet
  3. Input an invalid phone. You should see an error message saying "Invalid phone number"
  4. Input a voip phone (if you know of one). It should show a message saying that voip phones aren't allowed
  5. Input a valid phone. It should work
Request to merge fix/twilio-verification-message into master
The source branch is 2 commits behind the target branch
Open in Web IDE
Pipeline #119390498 passed for 1bb52381 on fix/twilio-verification-message
          Requires 3 more approvals from Devs, Deployers, and QA.
          Martin Santangelo Mark Harding Ben Hayward Emiliano Balbuena Rami Albatal
          Ready to be merged automatically. Ask someone with write access to this repository to merge this request

          Closes #1319

          Deletes source branch

          • Brian Hatchet :speech_balloon: @brianhatchet added Status::Requires Changes scoped label 4 minutes ago

            added scoped label

          Please register or sign in to reply
          0 Assignees
          None
          None
          Milestone
          None
          Time tracking
          No estimate or time spent
          1
          Labels
          Status::Requires Changes
          Lock merge request
          Unlocked
          9
          9 participants
          user avatar
          user avatar
          user avatar
          user avatar
          user avatar
          user avatar
          user avatar
          Reference: minds/engine!470