[jss-global] Link option does not work

[aslafy-z]

I'm unable to use the link option mentioned in the JS api documentation with jss-global plugin. Every call to sheet.update after sheet.attach have been called triggers a

Warning: Rule is not linked. Missing sheet option "link: true".

even with the right option, and sheet doesn't get refreshed.

I have been able to reproduce this behavior here: https://codesandbox.io/s/vml5lxmvy

Could be the same as #602 but op didn't said about the link option usage.

[kof]

Thank you for the report, I will take a look asap. In the mean time if you can, you can also try to fix it and send me pr or clues :)

[sambhav-gore]

@kof I have a solution for this, want to submit a PR but stuck with linting issues.
I never used flow before actually.

[kof]

Ok, send a PR with lint errors, I will take a look.

[sambhav-gore]

@kof I already created a PR with lint errors fixed - #666

[Vadorequest]

I have this warning with react-jss@8.2.1 and I have no idea what it may break or so.

[carlgunderson]

@kof is there an update on this issue based on #666? I'm getting the same warning but I'm not using an option called link anywhere, and can't find it mentioned in any docs...I'm using react-jss with jss-preset-default and this just started showing up. I don't have any dynamic properties inside my @global blocks.

[kof]

@carlgunderson as you can see that PR hasn't been worked on, I am still waiting for @sambhav-gore to finish it. Incase you don't have dynamic props and see this warning, please create a codesandbox demo

[jpetitcolas]

I encountered a similar issue with a non global scope (see details on mui/material-ui#10543 / sandbox: https://codesandbox.io/s/lpm6vop66l). May it be related?

[aslafy-z]

https://github.com/cssinjs/jss-global/pull/6 for reference

[aslafy-z]

This issue has been fixed with #1040.
Thanks @kof!