Kieran

> As far as I can tell the commit looks good. Lmk if you more testing pipeline stuff needs doing. > > ```shell > $ npm run test && npm...

> > As far as I can tell the commit looks good. Lmk if you more testing pipeline stuff needs doing. > > ```shell > > $ npm run test...

> @Kieran-McIntyre No rush. I jumped the gun a little with the PR anyways since I didn't add a test case for the complex obj. I'll do so later tonight...

> A side effect of this change is that the default sorting behaviour needs some rethinking. With the current assumption that `value` is a string the library uses it's first...

Thanks for submitting this @ahsanparacha02 @nicson0427. I'm looking into the implementation details, and I'm wondering what you think should happen if there are no items for a letter. Should the...