<Cmp error={...} /> is a namespace clash

I am using universal with great happiness, but I discovered an issue with the resulting component: it eats isLoading and error props, both pretty likely to be real component props.

Case in point, I’m wrapping react-select as a Field for redux-form, and having a from error results in the error being rendered instead of the component. It is very inconvenient to have to rename the error using 2 wrapper components 😦

I propose to namespace these props, e.g. universalLoading and universalError. This could be made optional?

[faceyspacey]

Can’t default to false. We need backwards compatibility. 3 + 4 minus defaulting to false.

PR idea approved

[wmertens]

Options:

1. put all props related to r-u-c under a prop: universal={{error, isLoading, etc}}
2. rename props individually: universalError, universalIsLoading
3. make props configurable: const Cmp = universal(()=>import('./Cmp'), {errorProp: 'universalError'})
4. same as 3, but make each prop also configurable as false if it should not be used, and default to false.

I think I prefer option 4. @faceyspacey if you think this is a good addition I’ll be happy to make a PR…