Skip to content

[ENHANCEMENT] [MER-5942] Migrate assessment attempt settings - #302

Open
andersweinstein wants to merge 2 commits into
masterfrom
MER-5942-migrate-maxAttempts
Open

andersweinstein wants to merge 2 commits into
masterfrom
MER-5942-migrate-maxAttempts

Conversation

@andersweinstein

@andersweinstein andersweinstein commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

Summary

Pass legacy assessment attempt settings through to Torus page resources.

  • Migrate max_attempts as maxAttempts.
  • Migrate recommended_attempts as recommendedAttempts.
  • Normalize legacy unlimited and -1 maximum-attempt values to Torus’s 0 sentinel.
  • Preserve Torus defaults when legacy values are absent or invalid.
  • Apply the settings to assessments and synthesized scored superactivity wrappers.

Testing

  • Added conversion and parsing coverage for numeric, unlimited, missing, and invalid values.
  • Full test suite, lint, and formatting checks pass.

MER-5942

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.

1 participant