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

Faith/mkt 634 extend composite filtering to plans tab #81

Merged
merged 2 commits into from
Jan 27, 2025

Conversation

fkomlo
Copy link
Contributor

@fkomlo fkomlo commented Jan 27, 2025

MKT-634

This is kinda just for the CMS rating filtering :)

Adds the option to pass in values for min, max, and step (increment) values for number inputs. This follows the same way that select inputs are receiving an array of { label + value } for the dropdown options. If not passed in, the step value defaults to 1 and max and min are ignored. The number options also are optional so that existing/future number inputs don't need to specify these settings.

Screenshot 2025-01-27 at 4 24 07 PM

@fkomlo fkomlo requested review from gnoha and avegancafe January 27, 2025 21:33
@avegancafe
Copy link
Contributor

Nice LGTM!

@fkomlo fkomlo merged commit 82694d4 into main Jan 27, 2025
2 checks passed
@fkomlo fkomlo deleted the faith/mkt-634-extend-composite-filtering-to-plans-tab branch January 27, 2025 22:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants