appsmith
appsmith copied to clipboard
[Bug]: Missing JS toggles in widgets and JSON Form columns
Is there an existing issue for this?
- [X] I have searched the existing issues
Description
The Default Country Code property in the Phone Input field has a JS toggle. However, this is missing inside the JSON Form.
Steps To Reproduce
Create a custom column for Phone Input inside a JSON Form. Notice that the Country Code property does not have a JS toggle.
Public Sample App
No response
Version
v1.8.1
Need to add JS for the following fields Phone input regular
- Allow country code change
Phone input json form
- Allow country code change
- Default country code
Currency input regular
- Allow currency change
- Decimals allowed
Currency input json form
- Default currency
- Allow currency code change
- Decimals allowed
@somangshu i would like to work on this issue
Greetings @flying-solo, thanks for showing interest 🎉 , This is all your. Assigning this to you now.
Please don't forget to read the Contribution Guidelines. Would appreciate if you can open a PR within the next 2 days. let us know here
Hey @somangshu, can I get some more information about the bug.
Sure, met me know what are the questions you have @flying-solo. This is happening in the phone input field of the json form.
cc @ashit-rath
@flying-solo some context
- In our property panes inside a widget, we have a js toggle for fields so that users can write js rather than selecting from a list. This helps developers make properties dynamic based on data or users.
- We need to enable this toggle in the fields i mentioned here - https://github.com/appsmithorg/appsmith/issues/16743#issuecomment-1246264850
- Please take a look at other fields in the widgets to know how this is being done. Like in the code go to the currencyInput widget and take a look at the properties inside it and see how to add JS toggle by looking at a property which already has a toggle
✌️cool, will work on it .
@flying-solo please let us know here if you have an open PR which is not yet tagged. We can take it forward
sorry for the delay i got a medical condition, you can un assign me i would contribute later
Sorry to hear that @flying-solo, Hope it better for you really soon. Would love to see you back here for contributions 😃
Heyy i would like to work on this. can u assign me?
Greetings @dubey-Ji, thanks for showing interest 🎉 , This is all yours. Assigning this to you now.
Please don't forget to read the Contribution Guidelines. Would appreciate if you can open a PR within the next 2 days. let us know here
Hey @dubey-Ji do you need help in it ?
Can i work on this issue ?
Greetings @trishitapingolia, thanks for showing interest 🎉 , This is all your. Assigning this to you now.
Please don't forget to read the Contribution Guidelines. Would appreciate if you can open a PR within the next 2 days.
Greetings @trishitapingolia, thanks for showing interest 🎉 , This is all your. Assigning this to you now.
Please don't forget to read the Contribution Guidelines. Would appreciate if you can open a PR within the next 2 days.
Thankyou for assigning me this issue, I'll be back with a pr soon.
Changes made only in widgets and not on JSON form in the PR mentioned in this issue https://github.com/appsmithorg/appsmith/pull/17725#issue-1415026071