appsmithorg / appsmith

Platform to build admin panels, internal tools, and dashboards. Integrates with 25+ databases and any API.
https://www.appsmith.com
Apache License 2.0
34.14k stars 3.7k forks source link

[Bug]: onTextChanged property throws Alert message even when incorrect input is entered on Phone widget #25911

Open chandannkumar opened 1 year ago

chandannkumar commented 1 year ago

Is there an existing issue for this?

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

somangshu commented 1 year ago

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 commented 3 months ago

HI @Nikhil-Nandagopal I am picking this issue.