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

Enhance SelectInput to return the selected choice onChange to set state on Wizard form review step #9850

Open
wattry opened this issue May 15, 2024 · 5 comments

Comments

@wattry
Copy link

wattry commented May 15, 2024

Is your feature request related to a problem? Please describe.
When using the select input we sometimes need to make a decision or add optionText as a display value.
Specifically when trying to set a display value for the review step of a Wizard form.
Autocomplete passes back the choice onChange, which allows us to populate a context provider with the corresponding display value for the optionValue, often times it is an id which is meaningless to the user.

Describe the solution you'd like
We'd like for the SelectInput and SelectArrayInput, preferably all choice inputs, to follow the same behavior of the autocomplete input by passing back the selected choice onChange events.

Additional context
Add any other context or screenshots about the feature request here.

Here is the documentation
AutoCompleteInput

AutocompleteArrayInput

@slax57
Copy link
Contributor

slax57 commented May 16, 2024

Sounds like a fair idea to me!
Feel free to open a PR if you'd like to give a try implementing this (remember to target the next branch if you do 🙂 ).

@wattry
Copy link
Author

wattry commented May 16, 2024

Sounds like a fair idea to me! Feel free to open a PR if you'd like to give a try implementing this (remember to target the next branch if you do 🙂 ).

Thanks, I'll see if I can get an opportunity to do so.

@wattry
Copy link
Author

wattry commented May 16, 2024

Sounds like a fair idea to me! Feel free to open a PR if you'd like to give a try implementing this (remember to target the next branch if you do 🙂 ).

Sounds like a fair idea to me! Feel free to open a PR if you'd like to give a try implementing this (remember to target the next branch if you do 🙂 ).

Sounds like a fair idea to me! Feel free to open a PR if you'd like to give a try implementing this (remember to target the next branch if you do 🙂 ).

I have discussed work with our PM and we cannot prioritize free time to put in a PR at this point in time.

Given we are enterprise users is there a way we can get this prioritized within the RA team, given this input does not conform to an enterprise component and it's documentation?

@slax57
Copy link
Contributor

slax57 commented May 17, 2024

@wattry Yes, you can request for a development to get prioritized via the Professional Services offer. As an Enterprise subscriber you are eligible to a 50% discount.

@fzaninotto
Copy link
Member

Adding a reference to #9245 (published in 4.14), where this was done for AutocompleteInput.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants