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

Fix: stale values in Cluster Settings #814

Merged
merged 1 commit into from
Sep 8, 2020

Conversation

aleksfront
Copy link
Contributor

Previously, inputs in Cluster Settings took initial values only in onComponentDidMount lifecycle which caused stale values after switching cluster withing settings page.

Adding autorun reactions to every field - those will update initial values in onComponentDidUpdate as well.

Fixes #809

Signed-off-by: Alex Andreev alex.andreev.email@gmail.com

Signed-off-by: Alex Andreev <alex.andreev.email@gmail.com>
@aleksfront aleksfront requested a review from a team September 7, 2020 09:21
@aleksfront aleksfront changed the title Refresh input values on cluster change Fix: stale values in Cluster Settings Sep 7, 2020
@nevalla nevalla added the bug Something isn't working label Sep 8, 2020
@nevalla nevalla added this to the 3.6.0 milestone Sep 8, 2020
Copy link
Contributor

@nevalla nevalla left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@aleksfront aleksfront merged commit a4eff11 into master Sep 8, 2020
@aleksfront aleksfront deleted the fix-update-data-switching-settings branch September 8, 2020 07:42
This was referenced Sep 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Wrong cluster name in cluster settings
2 participants