node-ts / bus

A typescript based enterprise service bus framework based on enterprise integration patterns

https://bus.node-ts.com/

MIT License

270 stars 26 forks source link

feat(#210): Support passing message and message context to Container Ad… #211

Closed manarhusrieh closed 9 months ago

manarhusrieh commented 10 months ago

210: Support passing message and message context to Container Adapter.

Add new parameter context to ContainerAdapter.get method.
Update the usages in bus instance and workflow registry to pass the message to get method.

adenhertog commented 9 months ago

You may also need to update bus-configuration.ts:261 to accept the same contract. I'm not sure why it's currently redeclared inline, I'm a bad programmer 😆 but the signature should be changed to:

withContainer(container: ContainerAdapter): this {

manarhusrieh commented 9 months ago

@adenhertog I have fixed the comments along with using ContainerAdapter interface. Please recheck it again.

Thanks

adenhertog commented 9 months ago

looks great @manarhusrieh , thanks!

adenhertog commented 9 months ago

released in https://www.npmjs.com/package/@node-ts/bus-core/v/1.0.17