[Bug]: Too restrictive typings for TomLoadCallback

[SWICRIOT]

Bug description

Examples for loading remote data shows the callback taking one and zero arguments. Typescript typings have defined TomLoadCallback to take two arguments : loadCallback(options: TomOption[], optgroups: TomOption[]): void;

Expected behavior

Typings should treat all parameters as optional.

Steps to reproduce

Use the following code in TypeScript

new TomSelect(accountSelect, {
    load: function (query: string, callback: TomLoadCallback)
    {
        var url = 'https://api.github.com/search/repositories?q=' + encodeURIComponent(query);
        fetch(url)
            .then(response => response.json())
            .then(json =>
            {
                callback(json.items);
            }).catch(() =>
            {
                callback();
            });
    }
});

This will show errors for the usage of callback.

Additional context

N/A

[nwalters512]

Can you please provide a self-contained reproduction on https://www.typescriptlang.org/play and call out specifically what errors you see? Thanks!

[JSteitz]

Typescript will complain about missing arguments when invoking the callback.

The documentation does not match the types.
Either the types need to be relaxed or the documentation needs to be adapted accordingly.

See also ticket #853