Skip to content

Fix: ranking round threshold validation#384

Merged
Jayprakash-SE merged 2 commits into
hatnote:masterfrom
Oyelakin-Mercy:fix/ranking-round-threshold-validation
Jun 27, 2026
Merged

Fix: ranking round threshold validation#384
Jayprakash-SE merged 2 commits into
hatnote:masterfrom
Oyelakin-Mercy:fix/ranking-round-threshold-validation

Conversation

@Oyelakin-Mercy

Copy link
Copy Markdown
Contributor

Fixes #383
What changed:

  • Added validation to prevent submitting a ranking round without selecting a threshold
  • Added montage-required-threshold i18n key for the error message

Before:
Screenshot 2026-02-27 004119


After:
threshold

@Oyelakin-Mercy Oyelakin-Mercy force-pushed the fix/ranking-round-threshold-validation branch from ef7203b to 05630ca Compare February 27, 2026 09:07

@lgelauff lgelauff left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This seems to be a working frontend fix. However, if you have the opportunity, it might be nice to also add a backend fix to at least give an informative error.

Comment thread frontend/src/i18n/en.json Outdated
@lgelauff lgelauff force-pushed the fix/ranking-round-threshold-validation branch from 05630ca to c11e300 Compare June 7, 2026 13:14
@lgelauff

lgelauff commented Jun 7, 2026

Copy link
Copy Markdown
Collaborator

I addressed myself. I believe this is now ready for approval

@Jayprakash-SE Jayprakash-SE merged commit ea1c1d3 into hatnote:master Jun 27, 2026
2 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Network error shown when creating ranking round without threshold selection

3 participants