You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository was archived by the owner on Feb 16, 2023. It is now read-only.
As things are defined right now, there's a slight risk that pages will not know about onlyExternal and do their own filtering, which would be less performant than using onlyExternal.
So if we switch to onlyExternal defaulting to true (or, more likely, switch its name to includeInternal and keep the default as false), there's a greater chance that developers will write code that has best performance.
However, it'd also be really annoying if essentially everyone has to override the default. So if we think that it's going to be far more common to want to be notified about same-connection changes, then we should probably leave things as-is.
Do we have a sense for what will be the most common behavior?
As things are defined right now, there's a slight risk that pages will not know about
onlyExternaland do their own filtering, which would be less performant than usingonlyExternal.So if we switch to
onlyExternaldefaulting totrue(or, more likely, switch its name toincludeInternaland keep the default asfalse), there's a greater chance that developers will write code that has best performance.However, it'd also be really annoying if essentially everyone has to override the default. So if we think that it's going to be far more common to want to be notified about same-connection changes, then we should probably leave things as-is.
Do we have a sense for what will be the most common behavior?