search

cartesi / rollups

Cartesi Rollups
30 stars 12 forks source link

Dispatcher Conditional Signer #56

Closed renan061 closed 1 year ago

renan061 commented 1 year ago

Tests passed locally (ran just now).

renan061 commented 1 year ago

Nice!

The only comment I have is that maybe some of the bureaucracy regarding signers and configuration could be on the tx-manager. But I'm not sure, maybe it's too specific to the dispatcher.

It is too specific. I tried it and it looked out of place.