-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Refactor of state_change functionality #11878
Conversation
b7723fb
to
1ec3e1d
Compare
I'm not sure if this is what I had to do. I left a question on the corresponding issue. |
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.
yep this is the first step, next step is removing the old file and updating the imports of where these functions are used
1ec3e1d
to
2ca887e
Compare
Got it, lmk what you think now. |
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.
this looks good ty,
but I don't want to include the extra ;
please undo these
671a69b
to
23b6a0b
Compare
Got it @mattsse, lmk what u think now mate. |
This PR is addressing the refactor specified on this issue: #11818