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

LO Event running TODO #103

Open
4 of 13 tasks
bradley-erickson opened this issue Oct 31, 2023 · 1 comment
Open
4 of 13 tasks

LO Event running TODO #103

bradley-erickson opened this issue Oct 31, 2023 · 1 comment
Labels
epoch Larger issue that includes subissues
Projects

Comments

@bradley-erickson
Copy link
Collaborator

bradley-erickson commented Oct 31, 2023

The LO Event package has been abstracted out, but there may be work left to do. Please leave any bugs, things not yet implemented, or other ideas in this issue.

  • Ajax logger
  • Opt in/out strategy
  • Add ability to specify which verbs, nouns, etc. we might use in the protocol in some kind of header (e.g. register our schema)
  • Fix and test queue.js within a node environment
  • Fix auto-increment within queue.js (currently resets to 0 each time)
  • Ensure all events have event field - Websocket's onclose and onerror do not
  • Implement blacklisting server side, test overall pipeline
  • Update server-side code to handle lock_fields, metadata, auth, etc.
  • Disable ability to send/store events if we are getting more than some number of exceptions
  • Resolve any storage errors when importing lo_event in another non-extension node project Uncaught TypeError: _storage_js__WEBPACK_IMPORTED_MODULE_0__.storage.get is not a function
  • We want to be able to fetch the websocketServer from storage
  • Add ability to specify util.backoff termination policy (do we just (1) die or (2) continue retrying using the last delay amount)
  • Consider structing / destructing in lo_event.js:init()
@bradley-erickson bradley-erickson added the epoch Larger issue that includes subissues label Oct 31, 2023
@bradley-erickson bradley-erickson added this to Icebox in Backlog Nov 30, 2023
@bradley-erickson
Copy link
Collaborator Author

bradley-erickson commented Dec 4, 2023

  • Adjust the authentication method to no longer throw an exception - use an if or while loop to accomplish
  • Update xAPI files (keep the nicely formatted ones, .json, and remove the originals)
  • Consider moving the xAPI download shell file into Python
  • All of xapi.cjs needs better function names, comments, etc. General code cleanup
  • Add test cases for all pathways and fallbacks for browserStorage.js
  • Document general logic flow for picking a storage item in browserStorage.js
  • Test disabler.js to ensure all use cases work correctly
  • Test how setFieldSet works when posted after INIT_READY in lo_event.js
  • Review small general improvements in util.js
  • Write good testing code for util.js
  • Review the firstConnection code in websocketLogger.js
  • Finish writing tests for queue.js
  • Review indexeddbQueue.js code
  • Consider stopping the event pipeline if we don't receive auth after X events

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
epoch Larger issue that includes subissues
Projects
Backlog
Icebox
Development

No branches or pull requests

1 participant