convert util TimeUtils to TS

[spasovski]

Was wondering if we could do a custom type for time as XX:XX:XX but otherwise pretty straightforward.

[mofojed]

I think with TypeScript, since the type is defined in the function itself, we don't need to include it in the docs anymore:

Suggested change

	* @param {integer} time in seconds
	* @param time in seconds

Granted integer isn't a valid type in TypeScript (or JS), only number is. So in this case, I think we should use a type alias to define a new type: https://www.typescriptlang.org/docs/handbook/2/everyday-types.html#type-aliases

We could just define type integer = number above, with a comment that we expect integers, then we can use the integer type here instead of number. Alternatively could name the type something time specific, like TimeInSeconds or something like that, but that may be getting too specific. I leave the name to your judgment.

[mofojed]

As you suggested in your comment, we can specify a better type using Type Literals: https://www.typescriptlang.org/docs/handbook/2/template-literal-types.html

In this case in particular, we can define a TimeString type, something like:

type TimeString = `${number}:${number}:${number}`;

Of course, that would match something like 0:4325423985:3928932, which is obviously not a valid time string. I don't know if there's an even better way to write it, but even defining a type alias with a comment of the format will be better.

[mofojed]

Looks good. In retrospect we should probably use Duration instead of Time to describe these things, but NBD