blazing-edge-labs / api-skeleton

3 stars 1 forks source link

Auto Batching Loaders #26

Closed rkatic closed 4 years ago

rkatic commented 4 years ago

TODO

rkatic commented 4 years ago

Not convinced with the new naming convention for "loader creators":

const loadSomethingT = createLoaderT(t => keys => {...})

const loadSomething = loadSomethingT(t)

loadSomething(1)

Instead of the "T", a more established and general postfix would probably be "With", however createLoaderWith would IMO become misleading.

Any idea for a better naming approach?

rkatic commented 4 years ago

To extend the idea with the "With" postfix:

const loadSomethingWith = loaderCreator(t => keys => {...})

const loadSomething = loadSomethingWith(t)

loadSomething(1)

In short, loadSomethingWith instead of loadSomethingT, and something like loaderCreator instead of createLoaderT.

The more generic "With" works great with custom "loader creators" that don't necessarly take (only) t..

Name loaderCreator remains a little strange.. which is usually the case for high order functions of 2nd+ order..