Remove spaceId from IRightPanelCardState[Stored]

[jryans]

It seems redundant, since the space ID is always the room ID.

[jryans]

@toger5 Any chance you remember why the spaceId field is there...? Perhaps I am missing a use case.

[toger5]

Not really. It was part of the old reference parameters. I think it is the same debate as with the member parameter of the card state. Either we have somewhat redundant parameters. Like roomMember and verificationUser or we only have member. One is redundant but more explicit and the other one. Well is not redundant but less explicit. I think we can do the same with spaceId and roomId (use the same param (roomId for both: spaceIdand roomId). I would at least add a comment to the state interface to make clear what the value is used for (for room and space id's)

[t3chguy]

At one point you could view the member list for a space without switching to the Space home/hierarchy, so it was needed for that, nowadays SpaceMemberList can probably go away entirely and just use RoomMemberList