-
Notifications
You must be signed in to change notification settings - Fork 87
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: changes for adding list attribute to the scale text field. #532
feat: changes for adding list attribute to the scale text field. #532
Conversation
…m/gaurishkamat/scale into feature/scale-text-field-datalist
Hi @acstll, Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is some code we don't need and some that can be simplified, I left some comments.
Thank you for the quick response!
packages/components/src/components/text-field/text-field.spec.ts
Outdated
Show resolved
Hide resolved
@gaurishkamat some checks are not passing, could you be so kind and try and fix those?
Thanks again 🙏 |
Thanks for the much needed detailed explanation @acstll. I have completed the above mentioned steps, let me know if anything is pending. |
@acstll not sure why the checks are failing. |
me neither. e2e pass locally for me. And Since e2e tests pass locally and the uncommitted changes seem to be OK too. I will merge. Thanks a lot again for your contribution. |
Hi @acstll should this be working for |
It should, the React package is automatically generated. Does it work? the latest |
No description provided.