Base solution for your next web application
Open Closed

User settings need to be saved before verifying phone number via SMS or error is returned. #6656


User avatar
0
jaycee26 created

Hi,

Just to report a potential confusing error/issue for users.

If a user goes to 'My Settings' and enters their phone number, the Verify Phone Number button to the right of the phone number field is selectable, but an 'unexpected error occured' is returned. When I looked in to this it is because the Phone Number at that point is not saved to the user profile, so the number used to send the SMS message to is null and (Twilio in my case) returns an error in the log saying that a 'To' number is required.

Therefore on 'My Settings' the user has to click 'Save' before verifying the phone number. Once saved, and the phone number is set in user.PhoneNumber, then the verification process works fine.

Could/should the user story be changed to only allow the 'Verify Phone Number' button to only be enabled once the phone number has been saved to the profile?

Thanks,


4 Answer(s)
  • User Avatar
    0
    ryancyq created
    Support Team

    Hi,

    • What is your product version?
    • What is your product type (Angular or MVC)?
    • What is product framework type (.net framework or .net core)?

    The current flow has implemented blocking of sms verificatiom when phone number was not saved before.

    Have you modified anything on the my-setting view? We can help to check.

  • User Avatar
    0
    jaycee26 created

    Hi @ryancyq

    I am using a relatively fresh copy of v6.7, AngulaJS, Core 2.2 with very little modification so far.

    I have not made any modifications to my-setting view. I have checked the code in my local source version against the current aspnet zero GitHub repo and the code is the same.

    Thanks.

  • User Avatar
    0
    ismcagdas created
    Support Team

    Hi @jaycee26

    Could you create the same issue on https://github.com/aspnetzero/aspnet-zero-core/issues so we can work on it for the next release.

    Thanks,

  • User Avatar
    0
    jaycee26 created

    Sure, I will do that now, thanks. I'm relatively new to aspnet zero so thought I better ask the question on here first! Thanks.