fix(types): RelayEnvironmentProvider should accept IEnvironment

[ianobermiller]

RelayEnvironmentProvider current accepts only the type Environment, which is actually the default export, or RelayModernEnvironment. Because of some funny naming by relay-runtime, we actually want "any environment", which is IEnvironment. This allows us to pass in a mock environment as opposed to a fully-fledged RelayModernEnvironment.

I also converted the component to a function for consistency and removed the generic since it isn't necessary.

[morrys]

hi @ianobermiller,
I'm sorry if I only replied now because I hadn't seen the pr :/

How about changing it like this?

export const RelayEnvironmentProvider = function <TEnvironment extends IEnvironment = Environment>(props: {

So the extends is the only thing that changes, this should solve your problem too.

[ianobermiller]

Yes, that would work, but the generic isn't necessary since there are no other props using it.

[morrys]

released with relay-hooks v9.0.0