Worldcubeassociation.org: Trainee delegates are listed on the Delegates page

Created on 27 Apr 2020  路  4Comments  路  Source: thewca/worldcubeassociation.org

According to @cubewhiz in #5118, Trainee Delegates are not (yet) WCA Staff and therefore should not be listed on the Delegate page.
It's probably just a matter of adding a .reject(&:trainee_delegate?) here.

I noticed this because the the "Trainee Delegate" badge has a link to the Delegates page, so ideally this badge should also be amended to not link there ^^

@danieljames-dj would you mind taking care of this please?

bug good first issue

Most helpful comment

Because Seniors change delegate statuses, not the WST (but it's not related to this specific issue actually).

The changes were merged yesterday, which is also when the email to Senior/WCAT was sent, so just be patient until the appropriate person actually change your status.

All 4 comments

Sure, I would be happy to. I've sent a PR. I hope this will fix the issues.

Ah, makes sense. I was curious why some Trainees were listed but others weren't.

Because Seniors change delegate statuses, not the WST (but it's not related to this specific issue actually).

The changes were merged yesterday, which is also when the email to Senior/WCAT was sent, so just be patient until the appropriate person actually change your status.

Hadn't come on github for a bit, I didn't catch that this is all fresh from yesterday :) Great job to everyone who worked on getting it completed!

Was this page helpful?
0 / 5 - 0 ratings

Related issues

Jambrose777 picture Jambrose777  路  3Comments

viroulep picture viroulep  路  6Comments

lgarron picture lgarron  路  9Comments

maconard picture maconard  路  7Comments

AlbertoPdRF picture AlbertoPdRF  路  4Comments