Skip to content

Conversation

@jamesh-gomo
Copy link
Member

@jamesh-gomo jamesh-gomo commented Aug 21, 2025

JIRA

https://getbridge.atlassian.net/browse/BRD-20938

Changes

  • Many updates to get tests passing (we were testing against lots of deprecated systems)
  • Bumped gem version from 0.9.0 to 0.10.0
  • This new version will be called from future versions of get_smart when the gem is repointed from 0.9.0 to 0.10.0
  • Set @api_version = 2 as default for ScormEngine interactions
  • Convert code to support ScormEngine API v2
  • Tested by test team on ScormEngine v24. Not tested on ScormEngine v20 (current Prod version). We can't rollback testing/staging
  • Lots of unit tests have been commented out. They were running against cassette responses generated with API v1. It will take too long to create cassette test data for this release before we need to go live. So that job has been postponed until after the release

Checklist

  • Feature Flag Required
  • Bug

Test plan

  • Walk-through
  • Peer review
  • Inspection
    • Detailed steps and prerequisites for validating the change:

Risk Analysis - the risk of change is evaluated

  • Low - Majority of the changes are low risk which doesn’t require extra testing, only code review by 1 reviewer
    • 1 reviewer
  • Medium - Some portion of changes are medium risk which need peer testing and review by 2 reviewers
    • 2 reviewers
    • peer testing
  • High - A very few breaking changes are high risk and need very throughout testing and review and also coordinated release process.
    • 2 reviewers
    • peer testing
    • coordinated release

jamesh-gomo and others added 30 commits August 21, 2025 16:50
@jamesh-gomo jamesh-gomo merged commit a369465 into master Oct 16, 2025
26 checks passed
@jamesh-gomo jamesh-gomo deleted the bug/BRD-20938-scormengine-v2 branch October 16, 2025 14:57
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.

3 participants