Suggestion: use object for input instead of tuple #99
Description
When passing in many inputs, it's awkward to remember the indexes inside the input tuple when you're trying to read/destructure a specific input, e.g:
useObservable(
input$ => {
const dispatch$ = input$.pipe(
map(input => input[5]),
distinctUntilChanged(),
);
/* … */
},
200,
[
props.shouldDisableInfiniteScroll,
props.isDoneFetching,
props.fetchDataParams,
props.fetchDataAndBuildActions,
props.dispatch,
props.history,
],
);If the inputs were represented as an object instead, they would be much easier to destructure.
useObservable(
input$ => {
const dispatch$ = input$.pipe(
map(({ dispatch }) => dispatch),
distinctUntilChanged(),
);
/* … */
},
200,
pick(
props,
"shouldDisableInfiniteScroll",
"isDoneFetching",
"fetchDataParams",
"fetchDataAndBuildActions",
"dispatch",
"history",
),
);samijaber, BlackHole1, denokon and BM-laoli