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

Add response to post save filter #11

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Commits on Jan 17, 2018

  1. Add alert toggle

    This creates an option in the settings page that allows alerts to either be displayed as an alert or written as console.log statements.  On a production site the error alerts could ruin user experience so this can be unchecked to give alerts without interferring with UX.
    KirkJohnson committed Jan 17, 2018
    Configuration menu
    Copy the full SHA
    6ff756a View commit details
    Browse the repository at this point in the history

Commits on Apr 3, 2018

  1. Configuration menu
    Copy the full SHA
    35b5f49 View commit details
    Browse the repository at this point in the history
  2. Adjusted code to match master

    Upstream modified changes on code so this modifies our code to work with the way they changed theirs.
    KirkJohnson committed Apr 3, 2018
    Configuration menu
    Copy the full SHA
    ac29eeb View commit details
    Browse the repository at this point in the history
  3. Add response to post-save-filter

    Adds lrs response to h5p-xapi-post-save filter so that if needing to store returned uuid for later voiding are modifying this is possible.
    KirkJohnson committed Apr 3, 2018
    Configuration menu
    Copy the full SHA
    6a8368d View commit details
    Browse the repository at this point in the history

Commits on Apr 4, 2018

  1. Configuration menu
    Copy the full SHA
    a324a98 View commit details
    Browse the repository at this point in the history