PR2189 enhanced phone information (International Number and validation) AND add phone extension

[dengcn]

Fixes #2189

Description

This is a feature delivery for followings

• Allow International Number for Phone Number, added validation accordingly
• Added phone extension and added validation accordingly
• Added new extension information to Summary
• Added new extension information to Final Analyst report

Please include a summary of the change. -regex -yupschema -contactform validation -summary report -Analyst report

Any new packages installed?

Give details

Required followup work

Is there anything related to this that still needs to be done (ex: documentation changes).

Checklist:

• [ ] I have updated the azurescript.sh with any new environment variables and added them to the appsettings
• [X ] I have looked at my code on GitHub and it all looks good (ex: no random commented out code or console.logs)
• [ ] I have added and needed tests for my changes (in particular for new screens)
• [ ] I have added a comment to any confusing code
• [ ] I have added documentation to any modified front-end code. (Or added missing documentation)

[lgtm-com[bot]]

This pull request introduces 1 alert when merging 37a06189ebdcae2b9f893dc3ea96c4b59051bf32 into 9297d3096c2eed44a182014309974d9a420506a4 - view on LGTM.com

new alerts:

• 1 for Duplicate character in character class

[dengcn]

FYI:

The followings are examples of good international number formats which will get validation pass. This had been communicated with Norm, Stephanie and Hong.

'00116131122323' //=> true '163454516131122323' //=> true '++1-7134445+345' //=> true '1.713.444.5345' //=> true '+09-323-333-3332' //=> true '853-12349543' //=> true '8534444' //=> true '(1)-(613)-(222-3333) //=> true

Checked with some international number specs, it requires Minimum 7 digits (some smallest country may fit into 7 digits).
+, -, . , () are allowed, and any character other than these 5 and 10 digits will fail to pass validation.

Extension number is also validated, only [0] to [9] are allowed, otherwise error message will show

[dengcn]

[dengcn]

From discussion with Stephanie and others in General section of TEAM, Phone validation is updated to ensure valid digits of phone should be more than or equal to 10 digits to ensure area code is captured. Max number is still 15 digits for international number use. Such code changes had been committed to GIT, and all test pass.