Skip to content

Fix: ranking round threshold validation#384

Open
Oyelakin-Mercy wants to merge 1 commit into
hatnote:masterfrom
Oyelakin-Mercy:fix/ranking-round-threshold-validation
Open

Fix: ranking round threshold validation#384
Oyelakin-Mercy wants to merge 1 commit into
hatnote:masterfrom
Oyelakin-Mercy:fix/ranking-round-threshold-validation

Conversation

@Oyelakin-Mercy

Copy link
Copy Markdown

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

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.

Network error shown when creating ranking round without threshold selection

3 participants