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] import of useEffect and useReducer in usePalette.tsx for SSR bundling #15

Merged
merged 1 commit into from
May 26, 2020

Conversation

JohnBra
Copy link
Contributor

@JohnBra JohnBra commented May 23, 2020

Includes tiny changes to imports of React hooks in usePalette.tsx
Fix according to this issue
This fixes #14

@coveralls
Copy link

Coverage Status

Coverage increased (+0.2%) to 92.105% when pulling 111c4cf on JohnBra:fix/imports-for-ssr-bundling into 62b7805 on leonardokl:master.

@NSpehler
Copy link

I can confirm that this fixes our issues with Gatsby, and doesn't seem to have negative downsides.

Would you be willing to tag this as 1.0.2 @leonardokl? Thank you for your time!

@leonardokl leonardokl merged commit 2272c2f into leonardokl:master May 26, 2020
@leonardokl
Copy link
Owner

Thanks for your contributions @JohnBra and @NSpehler .
I've published the version 1.0.2 with this fix.

@NSpehler
Copy link

Haven't done much, thanks a lot @JohnBra!

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.

Build time error when using inside Gatsby project.
4 participants