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

Read className from props #46

Merged
merged 1 commit into from
May 27, 2021
Merged

Conversation

robframpton
Copy link
Contributor

It's currently pulling className from state, but it's only defined in props.

@robframpton
Copy link
Contributor Author

Hey @nhunzaker, I know this repo hasn't had any changes in a long time, but would really appreciate it if we could merge and publish this small fix. If you don't have the bandwidth, please let me know. Thank you!

Copy link
Contributor

@nhunzaker nhunzaker left a comment

Choose a reason for hiding this comment

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

Oh, wild 🤦. Thanks for sending this in!

@cwmanning Hi :). I am no longer able to merge this PR or publish a new release. Would it be possible for someone on your end to do this? I think this should be a minor release to 6.5.0.

@cwmanning
Copy link
Member

Hey, sorry for the delay. I'll get this merged (plus one other open PR) and issue a minor release.

@cwmanning cwmanning merged commit b84c92b into vigetlabs:master May 27, 2021
@robframpton
Copy link
Contributor Author

Thank you both!

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.

3 participants