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]: Adding Regex validation in Currency Input throws error when input formats itself after loosing focus #20208

Open
1 task done
chandannkumar opened this issue Jan 30, 2023 · 1 comment
Assignees
Labels
Bug Something isn't working Currency Input Widget Issues related to currency input widget Medium Issues that frustrate users due to poor UX Needs Triaging Needs attention from maintainers to triage 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

Currency Input widget throws error when a regex validation is configured in property pane and when input in currency widget gets formatted automatically after loosing focus from widget

Steps To Reproduce

  1. Drag & Drop Currency Input widget
  2. Add a regex validation from property pane. Eg: ^\d+$
  3. Provide valid input in currency widget
  4. Observe that on clicking outside of the widget, the input gets auto formatted but throws validation error

Public Sample App

No response

Issue video log

https://bthrujcsw8.vmaker.com/record/SZnPivH5iVr2cyJ5

Version

Cloud / Production

@chandannkumar chandannkumar added Bug Something isn't working High This issue blocks a user from building or impacts a lot of users Production Needs Triaging Needs attention from maintainers to triage Currency Input Widget Issues related to currency input widget labels Jan 30, 2023
@github-actions github-actions bot added the Widgets Product This label groups issues related to widgets label Jan 30, 2023
@somangshu
Copy link
Contributor

Seems like the regex is working as expected.
Having a , in the formatting was a decision that was deliberately taken.

@dilippitchika there is direct conflict between two things here. What do you say is the way forward?

Also this is not a high priority issue, Adding , in your regex expression will make it work well enough.

@somangshu somangshu added Medium Issues that frustrate users due to poor UX and removed High This issue blocks a user from building or impacts a lot of users labels Feb 8, 2023
@Nikhil-Nandagopal Nikhil-Nandagopal added the Widgets & Accelerators Pod Issues related to widgets & Accelerators label Aug 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working Currency Input Widget Issues related to currency input widget Medium Issues that frustrate users due to poor UX Needs Triaging Needs attention from maintainers to triage 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

4 participants