Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug: Company Name and Phone fields accept misleading input #1047

Open
inesmrad opened this issue Apr 4, 2024 · 5 comments
Open

Bug: Company Name and Phone fields accept misleading input #1047

inesmrad opened this issue Apr 4, 2024 · 5 comments

Comments

@inesmrad
Copy link
Contributor

inesmrad commented Apr 4, 2024

Describe the bug
Company Name input field accepts numerical input and Phone Input field accepts alphabetical input.

To Reproduce
Steps to reproduce the behavior:

  1. Go to Companies
  2. Click on Add new Company
  3. Input just numbers in the company name field and letters in Phone field

Expected behavior
Company Name should only accept alphabetical input and Phone field should only accept numerical input.

Screenshots
Screenshot (649)
Screenshot (650)

Desktop (please complete the following information):

  • OS: Window 10
  • Browser : Chrome
  • Version : 123.0.6312.105
@tanmaytrivedi9140
Copy link

HI @inesmrad can u assign this to me
I would like to work upon it
Thank you

@inesmrad
Copy link
Contributor Author

inesmrad commented Apr 5, 2024

HI @inesmrad can u assign this to me I would like to work upon it Thank you

Sure, you can work on it

vr1Ravi added a commit to vr1Ravi/idurar-erp-crm that referenced this issue Apr 7, 2024
@vr1Ravi
Copy link

vr1Ravi commented Apr 7, 2024

Hello @inesmrad. I wanted to let you know that I've made some changes to the code to address the expected behavior. I've submitted a pull request for your review. Could you please take a look when you have a moment? Your feedback would be greatly appreciated.

@inesmrad
Copy link
Contributor Author

inesmrad commented Apr 7, 2024

Thanks for your work @vr1Ravi ! I'll try the PR and let you know

vr1Ravi added a commit to vr1Ravi/idurar-erp-crm that referenced this issue Apr 7, 2024
@dimple06varshney
Copy link

Hello @inesmrad If the issue is still open I want to work on this ticket. Please assign it to me.
Thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants