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

remove import of potentialFilterName from dimension-filters.ts and side-effect mutating it from toggleDimensionValueSelection #3798

Closed
bcolloran opened this issue Jan 8, 2024 · 2 comments
Labels
Type:Bug Something isn't working

Comments

@bcolloran
Copy link
Contributor

No description provided.

@bcolloran bcolloran added the Type:Bug Something isn't working label Jan 8, 2024
@bcolloran
Copy link
Contributor Author

Hey @briangregoryholmes and @ericpgreen2 , I didn't see #3728 until just now while cleaning up some other stuff.

Brian, there is no way you could have known this since it's unfortunately undocumented anywhere, but the intent is that all the actions and selectors in the state-managers folder should be pure function with no side effects, which importing the store potentialFilterName into dimension-filters.ts and updating it within the toggleDimensionValueSelection violates.

Not a big deal, and easy to rectify. I can take this issue if you want, or if you want to grab it that's fine too, just LMK :-)

@bcolloran
Copy link
Contributor Author

Obsolete following a recent PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type:Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant