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

When we create a web form and create a persons from here so in contact phone no. field is not taking validation, here added with letters also #1239

Open
komal-sh-27 opened this issue Apr 1, 2024 · 3 comments
Assignees
Labels
Bug Something isn't working. Fix Proposed When pull request is proposed for the issue.

Comments

@komal-sh-27
Copy link
Collaborator

Title:

When we create a web form and create a persons from here so in contact phone no. field is not taking validation, here added with letters also

Steps to Reproduce:

Step 1: Create a web form under settings option like this:

image

image

Step 2: Now create a persons from this web form with contact form add text instead of numbers:

image

image

Step 3: Now go to the admin panel under contacts -> persons option, here shows a text in phone no. coloumn:

image

Expected result:

If we have enter a text in phone no field then should be shown a warning like this:

The Contact Numbers field may only contain numeric characters

image

Actual result:

If we have added a text or characters in phone no field then web form submit without any warning shows.

@prabhat-webkul prabhat-webkul added the Bug Something isn't working. label Apr 10, 2024
@prabhat-webkul
Copy link
Collaborator

PR #1245

@prabhat-webkul prabhat-webkul added the Fix Proposed When pull request is proposed for the issue. label Apr 10, 2024
@harshit-webkul
Copy link
Collaborator

In web form page, The contact number field should only accept numeric values, and a validation message should appear if an alphanumeric or character value is entered.

Attached video/screenshot reference

issue.krayin-crm.1.mp4

@prabhat-webkul
Copy link
Collaborator

because basic validation is added for now as no custom js is used,

not planning currently, create a separate issue as enhancement/feature

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working. Fix Proposed When pull request is proposed for the issue.
Projects
None yet
Development

No branches or pull requests

3 participants