ucfopen / Obojobo

Next generation course content for your LMS. Easy for beginners, but powerful enough for researchers.
https://ucfopen.github.io/Obojobo-Docs/
GNU Affero General Public License v3.0
69 stars 34 forks source link

obojobo-express lti error constants make debugging difficult #912

Closed iturgeon closed 4 years ago

iturgeon commented 5 years ago

in obojobo-express/lti.js we're defining constant errors to make them easy to test identity and easy to read.

const ERROR_NO_OUTCOME_SERVICE_FOR_LAUNCH = new Error('No outcome service found for launch')

/* ... code ...*/

throw ERROR_NO_OUTCOME_SERVICE_FOR_LAUNCH

The problem is the callstack is relative to where the error is instantiated - which make's it useless, it's instantiated when the file is loaded.

I'd like to find another way to implement these that attempts to keep:

stale[bot] commented 4 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. We do this to keep our backlog under control, but we thank you for your contributions.