Skip to content

[stable5.4] fix(submission): do not require hidden questions, reject booleans on linear scales - #3877

Merged
Chartman123 merged 2 commits into
stable5.4from
backport/3870/stable5.4
Nov 11, 2026
Merged

Chartman123 merged 2 commits into
stable5.4from
backport/3870/stable5.4

Conversation

@backportbot

@backportbot backportbot Bot commented Nov 11, 2026

Copy link
Copy Markdown

Backport of PR #3870

The submit view hides questions without a title, choice, dropdown and
ranking questions without options, and grid questions without rows or
columns. The server still required an answer to such a question when it
was marked as required, so every submission failed with "Question "" is
required." and respondents could not fix it.

Skip the required check for these questions, using the same rules as the
submit view.

Assisted-by: Claude Code:claude-opus-5-5
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
A linear scale accepted the answer true as the value 1, as the check
cast it to the string "1". A list was cast to "Array" with a warning
before it was rejected. Both are now rejected as not being a value of
the scale.

Assisted-by: Claude Code:claude-opus-5-5
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
@backportbot backportbot Bot added bug Something isn't working 3. to review Waiting for reviews labels Nov 11, 2026
@codecov

codecov Bot commented Nov 11, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@Chartman123
Chartman123 disabled auto-merge November 11, 2026 10:16
@Chartman123
Chartman123 merged commit 9b7ff90 into stable5.4 Nov 11, 2026
51 checks passed
@Chartman123
Chartman123 deleted the backport/3870/stable5.4 branch November 11, 2026 10:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants