update rules + shouldShowNumberKeyboard logic #610
Conversation
|
Hi! I'm VTEX IO CI/CD Bot and I'll be helping you to publish your app! 🤖 Please select which version do you want to release:
And then you just need to merge your PR when you are ready! There is no need to create a release commit/tag.
|
|
Only one should be merged: #609 |
|
Your PR has been merged! App is being published. 🚀 After the publishing process has been completed (check #vtex-io-releases) and doing A/B tests with the new version, you can deploy your release by running:
After that your app will be updated on all accounts. For more information on the deployment process check the docs. 📖 |
Related to report:
https://vtexhelp.zendesk.com/agent/tickets/1098502
Rewrite the logic for shouldShowNumberKeyboard.
It was added a logic for NaN mask type to cover the use case of UK that can have different patterns like:
SW1A 1AA – Buckingham Palace, London
W1A 1AA – BBC Broadcasting House, London
EC1A 1BB – Central London
M1 1AE – Manchester
B1 1AA – Birmingham
So in order not to change the default behavior (not defining a mask should have the numeric keyboard) we return the alphanumerical board for mask defined like NaN.