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

feat(quantity-picker): Add the error message param on Quantity Picker #80

Closed
Tracked by #30
ManonPolle opened this issue Jan 30, 2023 · 0 comments · Fixed by #109
Closed
Tracked by #30

feat(quantity-picker): Add the error message param on Quantity Picker #80

ManonPolle opened this issue Jan 30, 2023 · 0 comments · Fixed by #109
Assignees
Labels
community 👥 As we stopped improvements for this version ofVitamin, this issue needs to be done by the community enhancement 🚀 New feature or request
Milestone

Comments

@ManonPolle
Copy link
Contributor

ManonPolle commented Jan 30, 2023

Is your feature request related to a problem? Please describe.
Error message has been added to Design Specificiations.
image

Describe the solution you'd like
Add optional error input param to add an error message under Quantity Picker.
Null Default value in order to have no breaking change to the component.

Additional context
Closes Decathlon/vitamin-design#30

@ManonPolle ManonPolle added enhancement 🚀 New feature or request run 💨 labels Jan 30, 2023
@corentin-stamper corentin-stamper self-assigned this Feb 3, 2023
@ManonPolle ManonPolle added this to the 0.3.0 milestone Mar 16, 2023
@lauthieb lauthieb added the community 👥 As we stopped improvements for this version ofVitamin, this issue needs to be done by the community label Mar 24, 2023
@lauthieb lauthieb removed the run 💨 label Apr 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community 👥 As we stopped improvements for this version ofVitamin, this issue needs to be done by the community enhancement 🚀 New feature or request
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants