HMTV2-12: creating room team table and relationships #4
Conversation
the idea is to have multiple judges and admins per room. during round 1 of judging, there may be more rooms with less judges and admins per room. in round 2, we don't need as many rooms, so we can have more judges and admins per room. right now i'm only assigning teams to rooms, but not to timeslots per room. will need to consider this.
test commit to see if i still get that pre-push error
jasutiin
requested review from
HarshitaJindia19,
YahyaAsmara,
anthonyych4n,
burtonjong and
skand088
as code owners
There was a problem hiding this comment.
maybe instead of dedicated tables for room admins and judges we look into instead having one Room table and creating references between admins, judges, and teams for the room.
There was a problem hiding this comment.
also look into the pipeline failing on this branch - run a biome check locally for syntax, ts, or formatting errors
There was a problem hiding this comment.
maybe instead of dedicated tables for room admins and judges we look into instead having one Room table and creating references between admins, judges, and teams for the room.
i'm not sure what you mean by this. do you mean having an admin and judge column inside one room table and putting their ids in there?
- admin: abc123
- judge: def456
the problem with this is it only allows one admin and judge per room, unless you meant storing them inside an array?
i just realized we have a role for each user, so another approach we could take is to remove one of the junction tables i have right now and keep one of them, and then rename it to judgingRoomStaff or something. another approach is that we could also put the room id inside the user table itself, but i don't like that idea lol.
but yeah lmk what you mean
2 participants
The idea is to create rooms for each round that we have. Each room can have a variable amount of admins and judges because:
I also made it so that instead of assigning teams to a round, it assigns teams to a judging room instead.