iov-core
iov-core copied to clipboard
Add tests to update BNS cash configuration
Each module can have a Configuration object and a transaction type to update this. For cash this is
Looks like there is only one configuration that can be updated at the moment
@webmaster128 Is it definitely just unit tests for this issue or do we need to sort out encoding/decoding of a transaction?
@willclarktech sorry, unittests -> tests. There should be a test showing that Configuration can be updated. This requires encoding/decoding of the ConfigurationMsg and the new configuration object.
Waiting for app.Tx to be updated, probably in Weave 0.17.1.
@webmaster128 What's the current status of this issue. Should we add it to 1.0?
@willclarktech I'd not consider it a blocker for 1.0.0, but as far as I can see it is ready for development. I moved it to position 1 in the backlog
Blocked by https://github.com/iov-one/weave/issues/815 which is scheduled for post-MVP.