Skip to content

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

Merged
rgachuhi merged 2 commits into
masterfrom
MER-5942-migrate-maxAttempts
Sep 21, 2026
Merged

rgachuhi merged 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

@rgachuhi
rgachuhi merged commit e3216a7 into master Sep 21, 2026
2 checks passed
@rgachuhi
rgachuhi deleted the MER-5942-migrate-maxAttempts branch September 21, 2026 13:33
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.

2 participants