Closed ximk closed 2 months ago
Sure it's a commonly requested feature. But it's somewhat complex feature so I'd first recommend starting with listing out all the changes you think you need to make and probably I can point out any potential shortcomings.
Hi! I am squishy. While I haven't worked in this codebase before, I work partially with typescript and html at my part-time job. Let me know if you want me to dive into the codebase for more specifics.
First of, some things to brainstorm:
enum
in order to differentiate between a regular competitive team and a special one. That would be regular, collegiate, special event (like national teams that only play for national events), LAN, school, and what else we could come up with. Having these in a centralized enum
will also make it easier to add to it later on with people's requests.Right now I was thinking about this very vague flow of changes I would implement, but this will change based on what we decide on the above questions:
Hmm based on your ideas here is how I think it could work:
AllTeamMember
table (integer, default 1)TeamMember
is implemented as a view currently. Maybe we should modify that view to also include the "is main team check". This would be good because then the assumption that one member has one team would stay true and you don't have to check every single place where it's used to make sure it can handle many teams. Then some new view ("TeamMemberIncSecondary" or whatever) that also includes the "secondary teams". Depending on the context you choose which you join with.
This would mainly be to give coaches the ability to appear on the team page of the team(s) they are coaching on top of the team they are already playing for (if they are on a team), but could also be applicable to players who are on two teams at once (maybe one for competitive and one for their collegiate team).
Squishy in the Discord said they'd be willing to work on it if this sounds like a good idea.