Jared Dykstra
Jared Dykstra
@srmagura - Happy to adopt all of your suggestions. Thanks for the quick response!
I've built out a couple of sites over the past few months, both using `react-loading-skeleton` and the `useSkeleton()` hook (submitted in this PR) provides a Developer Experience that has been...
@srmagura - I've squashed commits.
> I think [abstractions] belong in user-space, not in the library. If you do choose to add the `useSkeleton()` hook. (and I really hope you do!), it's very much an...
> @jared-dykstra Do you want to make a new PR that updates the README to show the `withSkeleton` pattern? I think we could show it as both a hook and...
> where does the module system fall short I don't think it does. I opened this PR to ensure that I'm using it correctly and making the most of it--I...
For example, I was expecting to have to setup Redux Devtools as an extra step, but was surprised to discover that it "just worked"
Here's a concise description of the problem I'm seeing. I suspect this is user-error (an error in my code) I currently have this code in place for HMR, and my...
The way I see it, there are a few things to do: 1. Each module needs to have it's own `module.hot.accept()` added to it, listening for changes in it's own...
Implementation is here: https://github.com/interbit/interbit/blob/master/packages/interbit-covenant-tools/src/mergeCovenants/index.js