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]: onTextChanged property throws Alert message even when incorrect input is entered on Phone widget #25911

Open
1 task done
chandannkumar opened this issue Aug 1, 2023 · 2 comments
Assignees
Labels
Bug Something isn't working Medium Issues that frustrate users due to poor UX Needs Triaging Needs attention from maintainers to triage Phone Input Widget Issues related to the Phone Input widget Production Widgets & Accelerators Pod Issues related to widgets & Accelerators Widgets Product This label groups issues related to widgets

Comments

@chandannkumar
Copy link

Is there an existing issue for this?

  • I have searched the existing issues

Description

Phone input only accepts number values hence the onTextChange property should only trigger when a phone input is updatedy.
But here we can observe that, even with typing alphabets, the event is triggered every time.. and event triggered all at once in JSONForm's phone input type.

Steps To Reproduce

  1. Drag & Drop Phone Input widget
  2. Configure onTextChanged property like {{showAlert('phone updated', 'success')}}
  3. Start typing the input as alphabets
  4. Observe that Alert appears on every input we type(doesn't matter correct or incorrect input)

Public Sample App

No response

Environment

Production

Issue video log

https://www.loom.com/share/f073f70b81574b29ad7c7e408790e8b4?sid=e348d96c-d3e3-46b9-85e2-528d3ce2174e

Version

Cloud/Production

@chandannkumar chandannkumar added Bug Something isn't working Production Needs Triaging Needs attention from maintainers to triage Phone Input Widget Issues related to the Phone Input widget Medium Issues that frustrate users due to poor UX labels Aug 1, 2023
@github-actions github-actions bot added the Widgets Product This label groups issues related to widgets label Aug 1, 2023
@chandannkumar chandannkumar changed the title [Bug]: onTextChanged property throws message even when correct input is not entered in Phone widget [Bug]: onTextChanged property throws message even when incorrect input is entered on Phone widget Aug 1, 2023
@chandannkumar chandannkumar changed the title [Bug]: onTextChanged property throws message even when incorrect input is entered on Phone widget [Bug]: onTextChanged property throws Alert message even when incorrect input is entered on Phone widget Aug 1, 2023
@somangshu
Copy link
Contributor

Similar issue observed before. @chandannkumar can you link those instances here.

@keyurparalkar I believe you had solved it earlier, maybe the same change is required here.

@SaiCharanChetpelly31
Copy link
Contributor

HI @Nikhil-Nandagopal I am picking this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working Medium Issues that frustrate users due to poor UX Needs Triaging Needs attention from maintainers to triage Phone Input Widget Issues related to the Phone Input widget Production Widgets & Accelerators Pod Issues related to widgets & Accelerators Widgets Product This label groups issues related to widgets
Projects
None yet
Development

No branches or pull requests

6 participants