Skip to content
New issue

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

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

Already on GitHub? # to your account

[376] Evaluator Dashboard #435

Merged
merged 10 commits into from
Mar 4, 2025
Merged

[376] Evaluator Dashboard #435

merged 10 commits into from
Mar 4, 2025

Conversation

stonefilipczak
Copy link
Contributor

Updated the evaluator dashboard to the latest designs.

Screen Shot 2025-02-20 at 10 07 04 AM Screen Shot 2025-02-20 at 10 06 50 AM

@r-bartlett-gsa r-bartlett-gsa added this to the Sprint 02/25/25 milestone Feb 20, 2025
@r-bartlett-gsa r-bartlett-gsa linked an issue Feb 20, 2025 that may be closed by this pull request
49 tasks
Comment on lines 18 to 19
<%= assigned_submissions_count(current_user, phase.challenge, phase) %> of
<%= remaining_evaluations_count(current_user, phase.challenge, phase) %>
Copy link
Contributor

Choose a reason for hiding this comment

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

Screenshot 2025-02-24 at 9 35 27 AM

the numbers are reversed and should be "# remaining of # assigned"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I agree that the numbers are reversed but I don't see that "# remaining of # assigned" copy on figma
https://www.figma.com/design/sLkJM1Ua9Zu3HW2h4mVdgr/Challenge.gov-Eval-(Updated-01%2F27%2F25)?node-id=7003-101843&p=f&t=DtwB5qvxhn4HKLYv-0

for me it looks like it should be "# of # Submissions"

Copy link
Contributor

Choose a reason for hiding this comment

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

yes sorry, that's what I meant. the "# remaining" is just the number itself, the string format you see on figma is correct. 👍

Copy link
Contributor

@stepchud stepchud left a comment

Choose a reason for hiding this comment

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

two issues to resolve

@stepchud
Copy link
Contributor

Can you also add some specs for this view (a11y and content validation)?

@stepchud stepchud mentioned this pull request Feb 24, 2025
Copy link
Contributor

@stepchud stepchud 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 to me!

@stepchud stepchud merged commit 9d7ee9b into dev Mar 4, 2025
10 of 11 checks passed
@stepchud stepchud deleted the 376_evaluator_dashboard branch March 4, 2025 19:24
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Dashboard Updates - Evaluator (Rails)
3 participants