Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New: Loss of Connection Feature (fixes: #268) - WIP #269

Closed
wants to merge 1 commit into from

Conversation

joe-replin
Copy link

@joe-replin joe-replin commented Mar 1, 2023

New

Solution for #268
Possible solution for #245
During a learner’s progression through a course, Adapt will be testing that connectivity is being maintained with the LMS. At the point a connection with either the LMS results in a failure, a notification pop-up will appear to the learner warning them of an error occurring and that future progression data isn’t being saved.

  • The learner will be allowed to retry establishing their connection or close the course.
  • Selecting the Retry button will cause Adapt to attempt sending and retrieving a small data file to and from the LMS’s server.
  • If unsuccessful, another notification will appear allowing the user to retry or close the module.
  • If unsuccessful a 2nd time, a final notification will appear informing the learner they must close and re-launch the course.
  • If successful, a new notification pop-up will appear informing the learner that their progress is being saved again.
  • Selecting the Close will close the course and learners will return to the LMS.
  • An additional configurable option will be provided if learners would like to continue progressing through the content at their own risk.
  • All SCORM progress is being saved up until the point at which the loss of connection occurred.

Testing

  1. More to follow

Work in Progress

While the connection error handling is in place, I still need to add in the UI for the notify pop-ups as well as configurable JSON fields to assist in on-screen messaging.

@joe-replin joe-replin self-assigned this Mar 1, 2023
@joe-replin joe-replin linked an issue Mar 1, 2023 that may be closed by this pull request

this._isInProgress = true;

// @todo: replace with fetch and polyfill for IE11?
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just use fetch, don't worry about the polyfill.

@joe-replin
Copy link
Author

New PR will be incoming later this week. Closing this one in the meantime.

@joe-replin joe-replin closed this Mar 6, 2023
@joe-replin joe-replin deleted the issue/268 branch March 6, 2023 19:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Loss of Connection Notification feature
2 participants