Matt Leathes

@paulfalgout thanks for clarifying the functionality, that is kinda what I suspected was going on TBH which is why I didn't assume it was a bug. I do think some...

Seems like a good addition to the wrapper code to me... at the moment anyone wanting to use the wrapper has to provide their own implementation ([here's](https://github.com/adaptlearning/adapt-contrib-spoor/blob/master/js/scorm/wrapper.js#L302-L308) how we do...

On the README for this repo it does say: > These wrappers are intended to make your life easier so you don't need to be a SCORM expert to add...

i’ll try and have a look over the next week

@canvasplay no sorry it's been a horribly busy few months! One thing that might help is if you had some unit tests showing that those functions have the correct output...

I suspect your version is probably correct. Like I say, most of our experience is with SCORM 1.2 so I'm certainly not claiming that our code is correct, more just...

@canvasplay @pipwerks please ignore everything I said about the SCORM 2004 time format - @canvasplay is absolutely correct to say 'format should be number+designator' i.e. PT4.3S and not PT4S.3S. Turns...

@pipwerks don't suppose you've had a chance to look at this have you? I think what @canvasplay has done would be a really useful addition...

👀

@pipwerks there are some unit tests in place if you want to verify it all behaves as it should...