Add useDataProviderMulti hook

[lyonsil]

This change is 

[tjcouch-sil]

Thanks!! I'm excited for this! My only request is a JSDoc comment :)

Reviewed 2 of 3 files at r1, 2 of 2 files at r2, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @lyonsil)

---

src/renderer/hooks/papi-hooks/use-data-provider-multi.hook.ts line 8 at r1 (raw file):

// eslint-disable-next-line @typescript-eslint/no-explicit-any
function useDataProviderMulti<T extends IDataProvider<any>[]>(

I think it would be good to have a JSDoc comment here. Also it would be good to copy over the comment explaining the rule disable from useDataProvider :) thanks!

Maybe worth mentioning the issue with types when you specify different data providers, too.

[lyonsil]

Reviewable status: 2 of 4 files reviewed, 1 unresolved discussion (waiting on @tjcouch-sil)

---

src/renderer/hooks/papi-hooks/use-data-provider-multi.hook.ts line 8 at r1 (raw file):

Previously, tjcouch-sil (TJ Couch) wrote…

I think it would be good to have a JSDoc comment here. Also it would be good to copy over the comment explaining the rule disable from useDataProvider :) thanks!

Maybe worth mentioning the issue with types when you specify different data providers, too.

Yep - I was so focused on the code I forgot about the JSDoc comments. Fixed.

[tjcouch-sil]

Looks great. Thanks for the hard work on this!

Reviewed 2 of 2 files at r3, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved