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 react-redux in kepler #63

Merged
merged 1 commit into from
Nov 7, 2019
Merged

fix react-redux in kepler #63

merged 1 commit into from
Nov 7, 2019

Conversation

Firenze11
Copy link
Contributor

Summary | Related Issue(s)

Kepler has the same react-redux issue as described in #59 . Did a quick fix based on methods discussed in keplergl/kepler.gl#351; long term fix involves upgrading react-redux version in Kepler (todo).

Changes

  1. Pass store directly to Kepler.

Test Plan

Tested Kepler is able to render in Manifold

Checklist

  • I have made this PR atomic.
  • I have provided enough context for others to review.
  • I have added tests to cover my changes (for features and bug fixes).
  • I have updated the documentation and changelogs accordingly.
  • All new and existing tests passed.

@Firenze11 Firenze11 merged commit a3dcb0c into master Nov 7, 2019
@Firenze11 Firenze11 deleted the fix-kepler-react-redux branch November 7, 2019 18:41
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.

None yet

2 participants