Skip to content
This repository was archived by the owner on May 28, 2021. It is now read-only.

Support React 16#33

Open
vicapow wants to merge 5 commits intomapbox:masterfrom
vicapow:master
Open

Support React 16#33
vicapow wants to merge 5 commits intomapbox:masterfrom
vicapow:master

Conversation

@vicapow
Copy link
Copy Markdown

@vicapow vicapow commented May 5, 2018

No description provided.

@vicapow
Copy link
Copy Markdown
Author

vicapow commented May 8, 2018

bump :)

@vicapow
Copy link
Copy Markdown
Author

vicapow commented May 9, 2018

cc: @jfirebaugh ?

@vicapow
Copy link
Copy Markdown
Author

vicapow commented May 11, 2018

sorry for being annoying. is this project abandoned?

@vicapow
Copy link
Copy Markdown
Author

vicapow commented May 11, 2018

I'm willing to maintain this project, if that helps

@emilymdubois
Copy link
Copy Markdown

Thank you for your patience @vicapow! We'll get you a code review this week.

romaxa and others added 2 commits May 15, 2018 13:13
@emilymdubois
Copy link
Copy Markdown

Hi @vicapow, thank you for your pull request and for your patience. I checked in with the team about a review, and after discussion, we decided that the best way forward is to rewrite this project from the ground up to better accommodate additional internal use cases. My suggestion would be to use your forked HOC for now until we are able to release a new package that's compatible with React 16.

@kepta should be opening an issue soon to start the revision discussion. Please let us know if you have any questions.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants