public async findPeopleWithId(personId: string): Promise<IPerson[]> { is awkward enough on its own, but what I only just realised is that a person may appear in multiple events with different roles.
This means that having the role on IPerson is pretty poor — I'm really not a fan of having multiple people per ID that correspond to the same physical human being acting in multiple capacities. Some denormalisation would be appreciated.
public async findPeopleWithId(personId: string): Promise<IPerson[]> {is awkward enough on its own, but what I only just realised is that a person may appear in multiple events with different roles.This means that having the
roleonIPersonis pretty poor — I'm really not a fan of having multiple people per ID that correspond to the same physical human being acting in multiple capacities. Some denormalisation would be appreciated.