Skip to content

Handle visibility for events #701

Description

@Shatur

Right now events don't take visibility into account. If you use Broadcast, this will send the event and log an error on clients that don't see the entity on deserialization. But we can fix that: all events that touch entities implement the MapEntities trait and registered in a special way. We could use this to do "fake" entity mapping that actually just collects the entities touched by the event into a Vec. Then, before sending, we check whether each entity is visible (we have an API for it).

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions