appsmith icon indicating copy to clipboard operation
appsmith copied to clipboard

[Bug]: Missing JS toggles in widgets and JSON Form columns

Open danciaclara opened this issue 3 years ago • 15 comments

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. Screenshot 2022-09-14 at 11 04 19 Uploading Screenshot 2022-09-14 at 11.05.01.png…

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

danciaclara avatar Sep 14 '22 05:09 danciaclara

Need to add JS for the following fields Phone input regular

  1. Allow country code change

Phone input json form

  1. Allow country code change
  2. Default country code

Currency input regular

  1. Allow currency change
  2. Decimals allowed

Currency input json form

  1. Default currency
  2. Allow currency code change
  3. Decimals allowed

dilippitchika avatar Sep 14 '22 05:09 dilippitchika

@somangshu i would like to work on this issue

whiletrueee avatar Sep 15 '22 04:09 whiletrueee

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

somangshu avatar Sep 15 '22 05:09 somangshu

Hey @somangshu, can I get some more information about the bug.

whiletrueee avatar Sep 15 '22 05:09 whiletrueee

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

somangshu avatar Sep 15 '22 06:09 somangshu

@flying-solo some context

  1. 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.
  2. We need to enable this toggle in the fields i mentioned here - https://github.com/appsmithorg/appsmith/issues/16743#issuecomment-1246264850
  3. 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

dilippitchika avatar Sep 15 '22 06:09 dilippitchika

✌️cool, will work on it .

whiletrueee avatar Sep 15 '22 06:09 whiletrueee

@flying-solo please let us know here if you have an open PR which is not yet tagged. We can take it forward

somangshu avatar Sep 21 '22 10:09 somangshu

sorry for the delay i got a medical condition, you can un assign me i would contribute later

whiletrueee avatar Sep 21 '22 10:09 whiletrueee

Sorry to hear that @flying-solo, Hope it better for you really soon. Would love to see you back here for contributions 😃

somangshu avatar Sep 21 '22 10:09 somangshu

Heyy i would like to work on this. can u assign me?

dubey-Ji avatar Oct 08 '22 18:10 dubey-Ji

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

dilippitchika avatar Oct 10 '22 05:10 dilippitchika

Hey @dubey-Ji do you need help in it ?

Vishrut19 avatar Oct 11 '22 19:10 Vishrut19

Can i work on this issue ?

trishitapingolia avatar Oct 17 '22 17:10 trishitapingolia

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.

somangshu avatar Oct 18 '22 09:10 somangshu

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.

trishitapingolia avatar Oct 19 '22 09:10 trishitapingolia

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

chandannkumar avatar Nov 11 '22 05:11 chandannkumar