"Using `null` as a rangeBehavior value is deprecated" when using "ignore"

[miracle2k]

I implement rangeBehaviours as a function and return "ignore". Relay shows me this warning:

Usingnullas a rangeBehavior value is deprecated. Useignoreto avoid refetching a range.
Callsite:

relay/src/traversal/writeRelayUpdatePayload.js

Line 416 in 712f592

if (!rangeBehavior || rangeBehavior === IGNORE) {

Is the condition there wrong? I'm thinking it might need to be (!rangeBehavior && rangeBehavior !== IGNORE)

[xuorig]

Oops, good find @miracle2k. The condition was shared because we also want to early return when it's IGNORE so (!rangeBehavior && rangeBehavior !== IGNORE) would probably not be what we want here.

PR coming for this!