-
-
Notifications
You must be signed in to change notification settings - Fork 3
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Added more APIs to store and fixed bugs
- Loading branch information
Showing
5 changed files
with
515 additions
and
265 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,39 @@ | ||
# Code Requests | ||
|
||
- Support for undo-ing an action (This will need the implementation of an undo stack) | ||
- define **Radixx.attachMiddleware(function: fn)** | ||
|
||
```js | ||
|
||
// below is the form of the implementation | ||
|
||
Radixx.attachMiddleware(function(aciton, next){ | ||
if(action.actionData === null){ | ||
action.actionData = [{}]; | ||
} | ||
return next(); | ||
}); | ||
``` | ||
|
||
- modify **Radixx.createAction(object: obj)** | ||
|
||
```js | ||
|
||
// below is the form of the implementation | ||
Radixx.createAction({ | ||
// used to identify an action | ||
'doThing':{ | ||
type:'DO_THING', | ||
// define what the payload for this action should be and look like | ||
payloadDefinition:[ | ||
Radixx.Payload.typing.array.isRequired(), | ||
Radixx.Payload.form.object.asPart() | ||
] | ||
} | ||
}); | ||
``` | ||
|
||
- Optimize <q>onDispatch</q> event not to fire only when any store data ACTUALLY changes | ||
|
||
- See if there's a way to stop the propagation of _'storage'_ events whenever **Radixx** is making use of sessionStorage | ||
|
||
- Detect memory leaks and clean them up. |
Oops, something went wrong.