Closed paquettealice closed 4 years ago
In the middle of rebasing but have to go eat, so things will look weird for a bit :p
Rebasing should be done, and I made two changes:
ICorrelatedAction
is now exported as part of the public apidist/...
now imports from @briebug
instead.Please let me know if you want additional changes!
@jrista Pushed the changes to the versions, check them out when you can.
This introduces the new
correlationId
property for allEntityActions
. The changes to the library itself are spread wide but quite shallow, so it shouldn't be too bad to review. The demo app changes are quite heavy however, and I unfortunately don't have time for a full write-up. The broad strokes for both are as follow:Library
uuidv4
ICorrelatedAction
with just one property:correlatedId
IEntityAction
now extends bothAction
andICorrelatedAction
EntityAction
class constructor now accepts an optional 3rd argument:public correlationId: string = uuid()
EntityAction
now accept an optional 3rd argument:correlationId?: string
which is passed tosuper()
correlationId
to subsequentSuccess
andFailure
actions, linking those sets of actions togethercorrelationId
argument to the BaseFacade's dispatch methods (and pass it down the chain like it's done above)? E.g.load
,loadMany
,create
,update
, etc.Demo
If you have any questions, don't hesitate to ask!
P.S. I didn't add any unit tests for the library changes (I completely forgot and I'm crazy strained for time lately), so if you'd rather wait until I can get those up before merging, that's completely fine.