Skip to content

HMT-63 Judging Dashboard #47

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 14 commits into from
Jul 11, 2024
Merged

HMT-63 Judging Dashboard #47

merged 14 commits into from
Jul 11, 2024

Conversation

hannagracec
Copy link
Contributor

@hannagracec hannagracec commented Jun 20, 2024

Added the dashboard for judges to display the teams that are assigned to their room. Includes stats panels displaying number of teams assigned to their room and number of teams left to score. The scoring table displays the team names, has a column for each scoring component, has a sort button to switch between alphabetical and reverse alphabetical order, and includes pagination.

The scoring table can be reused for the Admin side. There are props to edit the colour scheme (either pink or purple) and determine how many entries are shown per page (since there are more entries shown to admins).

image

A popup opens when clicking the create score button displaying the team name.
image

Similarly, with the edit score button, the team name will be displayed.
image

These popups can be changed once the scoring flow is made!

@hannagracec hannagracec requested a review from ideen1 June 20, 2024 21:04
Copy link
Contributor

@ideen1 ideen1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good overall, one question. I think I put all the right data in my tables but there wasn't an "add score" button that appeared when I had a team left to score, the table was just empty(not even the teams assigned to me showed up). Did I maybe forget to add something to one of my tables?

@hannagracec
Copy link
Contributor Author

@ideen1 hmm that's weird, did anything show up in the console? If not, it does use quite a few different tables so just to double check there should be items in the teams, teamRoom, room, and Hackathon tables.

@ideen1 ideen1 self-requested a review July 3, 2024 00:51
@ideen1
Copy link
Contributor

ideen1 commented Jul 3, 2024

@hannagracec Status update: still on my list will be testing shortly.

Copy link
Contributor

@ideen1 ideen1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, to be tested in master after additional features are added

@hannagracec hannagracec merged commit b395ab8 into main Jul 11, 2024
4 checks passed
@hannagracec hannagracec deleted the HMT-63-judging-dashboard branch July 11, 2024 17:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants