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(Canvas): Remove mobx warnings from @patternfly/react-topology #1295

Merged
merged 1 commit into from
Aug 8, 2024

Conversation

lordrip
Copy link
Member

@lordrip lordrip commented Aug 8, 2024

Context

When opening and closing the configuration tab, mobx warnings are thrown from the @patternfly/react-topology library.

The fix is to wrap any operation to the canvas inside of an action callback, so mobx can notify the consumers that something changed.

fix: #1294

When opening and closing the configuration tab, mobx warnings are
thrown, coming from the @patternfly/react-topology library.

The fix is to wrap any operation to the canvas inside of an `action`
callback, so mobx can notify the consumers that something changed.

fix: KaotoIO#1294
@lordrip lordrip force-pushed the fix/mobx-warnings branch from 82df317 to de80a9f Compare August 8, 2024 11:24
@lordrip lordrip merged commit c3eb9ee into KaotoIO:main Aug 8, 2024
9 checks passed
@lordrip lordrip deleted the fix/mobx-warnings branch August 8, 2024 11:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Interacting with the Canvas generate mobx warnings
2 participants