Merge pull request #8 from expertlead/hao/nap/dispose

Hao/nap/dispose

CHANGELOG.md

@@ -1,3 +1,6 @@
+*v2.0.5*
+_Feat: expose nap disposer function as a third arg to the nap function_
+
 *v2.0.4*
 _Feat: implement nap for state_
 

examples/todo/state.jsx

@@ -19,9 +19,10 @@ const aggregateAuthenticatedUser = ({ user }) => ({
   userFullname: user.username,
 })
 
-const userStateNextActionPredicate = (state, onlyTrackedModels) => {
+const userStateNextActionPredicate = (state, onlyTrackedModels, dispose) => {
   if (state.isAuthenticated) {
     console.log("User logged in, send notification to service x")
+    dispose()
   }
 }
 

package.json

@@ -1,10 +1,12 @@
 {
   "description": "Functional Reactive State Management Library for React, using MobX, implementing the SAM pattern",
   "name": "samx",
-  "version": "2.0.1",
+  "version": "2.0.5",
   "main": "src/index.js",
   "repository": "[email protected]:expertlead/samx.git",
   "author": [
+    "Hamza Ouaghad",
+    "Leonardo Kewitz",
     "Invisible Technologies",
     "Expertlead"
   ],

src/index.js

@@ -1,6 +1,6 @@
 import { configure } from 'mobx'
 
-configure({ enforceActions: 'always', isolatedGlobalState: true })
+configure({ enforceActions: 'strict', isolatedGlobalState: true })
 
 export * from 'mobx'
 

src/state.js

@@ -38,7 +38,7 @@ const State = (transformState, nap = undefined) => ({
   let retrieveModels = getModels
 
   if (onlyTrack) {
-    
+
     retrieveModels = () => {
       onlyTrack.forEach(
         (trackedModel) => {
@@ -69,13 +69,15 @@ const State = (transformState, nap = undefined) => ({
   )
 
   if (nap) {
-    const nextActionPredicateEffectFn = (stateRepresentation) => (models) => () => {
-      nap(stateRepresentation, models)
+    const nextActionPredicateEffectFn = (stateRepresentation) => (models) => (disposer) => () => {
+      nap(stateRepresentation, models, disposer)
     }
 
-    reaction(
+    const dispose = reaction(
       retrieveDataFn,
-      nextActionPredicateEffectFn(stateRepresentation)(retrieveModels())
+      nextActionPredicateEffectFn(stateRepresentation)(retrieveModels())(
+        () => dispose()
+      )
     )
   }