-
Notifications
You must be signed in to change notification settings - Fork 42
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
Performance: Update inputs in flow settings - parameters #1103
Conversation
I'm seeing the same issue here with type not resetting as mentioned on #1095. I think if we're going to show type we need to fully reset the type when reset is clicked. |
For info - I'm going to hold off on reviewing this one further until changes in #1095 are made and merged so I'm not commenting on the same things across PRs. |
2d4ba07
to
3c7a244
Compare
3c7a244
to
ecb1559
Compare
ecb1559
to
aa486f5
Compare
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.
LGTM @stackoverfloweth
NOTE: until #1095 is merged this will appear much larger than it actually is. Only the last commit is different