#1146 · open · 6 comments
<img width="709" height="473" alt="Image" src="https://github.com/user-attachments/assets/d6bf6d8c-3f37-4648-b6ae-b3ab7f5b59b0" /> As attached in above image, user is able to input text in phone field. Phone field should only accept numbers.
Hi @hishamco , I will be raising the PR for the above issue. Hope this is a valid bug.
https://github.com/simplcommerce/SimplCommerce/pull/1147 this is the PR for the above fix. Please review and approve.
Does the `PhoneAttribute` accept letters?!!
No @hishamco , Phone field will not accept letters after my PR fix.It only accepts numbers. The screenshot showing Phone field accepting letters is to log a bug which was happening before raising the PR.
That's what I'm expecting, the simple fix to change the input type to `number`
> That's what I'm expecting, the simple fix to change the input type to `number` Yes @hishamco , Please find the below PR for the input type to number, please review and add comments , will modify according to your review and will update the changes if required. https://github.com/simplcommerce/SimplCommerce/pull/1147 <img width="919" height="385" alt="Image" src="https://github.com/user-attachments/assets/9a6736b2-e278-4bf8-ad6d-02a13b97e1cc" />