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

Breaking: ES6 Upgrade (fixes: #13) #30

Closed
wants to merge 15 commits into from
Closed

Breaking: ES6 Upgrade (fixes: #13) #30

wants to merge 15 commits into from

Conversation

joe-replin
Copy link
Contributor

@joe-replin joe-replin commented Feb 28, 2024

Update

  • Removing favorite, unfavorite & modifying preferred statements
  • ES6 Upgrade of all JS

New

  • New verb accessed for resource clicks
  • New verb preferred for visua11y clicks
  • Adding iterative attempt numbers to question statements

Breaking

  • This upgrade moves to use updated core imports instead of the redundant behavior prior to v5.19.1
  • fixes es6 upgrade #13

Testing

  1. MUST have authorization credentials to connect to an established LRS. These credentials can be manually inputted into required/launch.html.
  2. Enable the plug-in in config.json. See example.json.
  3. Navigate through a vanilla course, checking that each statement is appropriately being recorded in the correct store within the LRS.

@joe-replin joe-replin self-assigned this Feb 28, 2024
@joe-replin joe-replin closed this Feb 29, 2024
@oliverfoster
Copy link
Member

oliverfoster commented Mar 1, 2024

What happened to this? It is definitely needed.

@joe-replin
Copy link
Contributor Author

I was attempting to put my WIP within a PR. Didn't want people to think it was finished and I don't want to spam people with incessant emails. So I'm working within the branch first until I'm good & ready.

@oliverfoster
Copy link
Member

Ok. Good plan.

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.

es6 upgrade
2 participants