-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Added missing isEnabled
command in new Element API
#4041
Added missing isEnabled
command in new Element API
#4041
Conversation
isEnabled
command in new Element API
Hey @garg3133 , is there any changes needed? please let me know |
Same comment as posted on #4039: #4039 (comment) |
Hey, I have made some changes. |
Hey @garg3133 also I executed the test and type test. |
Hey @garg3133 , I have made certain changes and worked according to the previous pr. |
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.
Looks good now!
Cool, sorry for making those silly mistakes. |
Thanks in advance for your contribution. Please follow the below steps in submitting a pull request, as it will help us with reviewing it quicker.
fix - #4032