Merge pull request cerebral#38 from FWeinb/fix-type-order

refactor(overmind): switch order of Effects and State on ActionsCallb…

Author: christianalfoni

packages/node_modules/overmind/src/index.ts

@@ -94,7 +94,7 @@ export interface Events {
   }
 }
 
-export type ActionsCallback<Effects, State> = (
+export type ActionsCallback<State, Effects> = (
   action: TAction<State, Effects & { state: State }>
 ) => any
 
@@ -122,15 +122,15 @@ export default class App<
         [namespace: string]: ReactionsCallback<State, any>
       },
   Actions extends
-    | ActionsCallback<Effects, State>
+    | ActionsCallback<State, Effects>
     | {
         [namespace: string]: ActionsCallback<{}, {}>
       }
 > {
   private proxyStateTree: ProxyStateTree
   eventHub: EventEmitter<Events>
   devtools: Devtools
-  actions: Actions extends ActionsCallback<Effects, State>
+  actions: Actions extends ActionsCallback<State, Effects>
     ? ReturnType<Actions>
     : Actions extends {
         [namespace: string]: ActionsCallback<{}, {}>
@@ -345,7 +345,7 @@ export default class App<
   private getActions(configuration, action) {
     const actions =
       typeof configuration.actions === 'function'
-        ? (configuration.actions as ActionsCallback<Effects, State>)(
+        ? (configuration.actions as ActionsCallback<State, Effects>)(
             action as TAction<State, Effects & { state: State }>
           )
         : (Object.keys(configuration.actions || {}).reduce(

packages/node_modules/overmind/src/views/react.ts

@@ -31,9 +31,9 @@ export default class ReactApp<
   Reactions extends ReactionsCallback<State, any>,
   Actions extends
     | {
-        [namespace: string]: ActionsCallback<Providers, State>
+        [namespace: string]: ActionsCallback<State, Providers>
       }
-    | ActionsCallback<Providers, State>
+    | ActionsCallback<State, Providers>
 > extends App<State, Providers, Reactions, Actions> {
   connect = <
     Props,