Skip to content
This repository was archived by the owner on Feb 16, 2023. It is now read-only.
This repository was archived by the owner on Feb 16, 2023. It is now read-only.

Should onlyExternal use a different default? #21

@sicking

Description

@sicking

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?

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions