the-platform icon indicating copy to clipboard operation
the-platform copied to clipboard

Broken with 16.8.0-alpha.0

Open VinSpee opened this issue 5 years ago • 15 comments

Hi!

As is expected for us bleeding edge-riders, [email protected] breaks this package. I think it has something to do with react-cache, maybe updating it to 2.0.0-alpha.1 would help?

barebones repro: https://codesandbox.io/s/oqy12yrk65

readContext https://oqy12yrk65.codesandbox.io/node_modules/react-cache/2.0.0-alpha.0/cjs/react-cache.development.js:248:21

Object.read https://oqy12yrk65.codesandbox.io/node_modules/react-cache/2.0.0-alpha.0/cjs/react-cache.development.js:316:7

Img https://oqy12yrk65.codesandbox.io/node_modules/the-platform/Img.js:26:15

EDIT: Yeah this is a react / react cache problem. They've updated the react-cache repo, but haven't released the package. see https://github.com/facebook/react/issues/14558

VinSpee avatar Jan 14 '19 15:01 VinSpee

@jaredpalmer any word on this? I tried upgrading the packages in yesterday in a fork, but they were still broke from react-cache. Not sure how to help get this resolved as I can't seem to find any docs around react-cache. 🤷‍♀️

It's affecting all the components.

caseybaggz avatar Jan 30 '19 14:01 caseybaggz

@caseybaggz there is nothing practical to be done until React-cache releases an update.

VinSpee avatar Jan 30 '19 14:01 VinSpee

@VinSpee they've already released it. 2.0.0-alpha.1 is live.

caseybaggz avatar Jan 30 '19 17:01 caseybaggz

@caseybaggz image

2.0.0-alpha.1 was an old release which happened before the currentOwner -> currentDispatcher changes which are causing this issue.

VinSpee avatar Jan 30 '19 17:01 VinSpee

@caseybaggz image

2.0.0-alpha.1 was an old release which happened before the currentOwner -> currentDispatcher changes which are causing this issue.

Gotcha. My bad. I miss-interpreted your original message. 👍

caseybaggz avatar Jan 30 '19 17:01 caseybaggz

A similar issue exists with [email protected]. My ecomm project is all kinds of broken right now.

Hope we can get a new update of the react-cache soon...

IadityaM avatar Feb 11 '19 20:02 IadityaM

the master branch already contains the fix. @jaredpalmer can we have an npm release plz?

dreyks avatar Feb 12 '19 16:02 dreyks

kraken

vitordino avatar Feb 23 '19 02:02 vitordino

0.10.0 released 🎉

thx @jaredpalmer

dreyks avatar Feb 27 '19 21:02 dreyks

yay!

VinSpee avatar Feb 27 '19 21:02 VinSpee

does this now require vendoring react-cache?

VinSpee avatar Feb 27 '19 21:02 VinSpee

i think not vendoring but just adding it to the package.json on your own

https://github.com/palmerhq/the-platform/pull/71/commits/31a119fc73f3a7f4bbcc34eafe8edcfd67c9325c

dreyks avatar Feb 27 '19 21:02 dreyks

https://codesandbox.io/s/o7mk6p274y

example codesandbox is still complaining about dispatcher

VinSpee avatar Feb 27 '19 21:02 VinSpee

true... adding react-cache@^2.0.0-alpha.1 does not solve the issue

dreyks avatar Feb 27 '19 21:02 dreyks

figured things out finally: since react-cache does not have a published package compatible with react 16.8+ you have to build it from master and vendor it. this is exactly what was done in the-platform so you can just grab it from https://github.com/palmerhq/the-platform/tree/c1d20b31a49ab5df01f21448e2e6129ffd51a57e/example/vendor/react-cache

dreyks avatar Mar 11 '19 21:03 dreyks