Missing MapRedeemerKeyToRedeemerVal as default Redeemer

[solidsnakedev]

Currently, the default transaction builder relies on the LegacyRedeemer, but the new hard fork enforces the redeemer to be a map.

https://github.com/IntersectMBO/cardano-ledger/blob/397bf8fa1ebf94c3bc962a1a14e73f0c30453b09/eras/conway/impl/cddl-files/conway.cddl#L343-L345

[rooooooooob]

@solidsnakedev do/will you have a fix yourselves or should I work on one?

[solidsnakedev]

I'd be great if you can, happy to help doing some preview testing on my end

[rooooooooob]

@solidsnakedev let me know if #368 works for you

[solidsnakedev]

that was pretty fast, thank you! will do some tests soon

[rooooooooob]

@solidsnakedev which network exactly has been upgraded? is it the regular preview that's been changed? or another preview network that has been spun up?

[solidsnakedev]

Both preprod and preview already forked to Chang 2.
AFAIK the old transaction format is still valid, but to be honest, I'm not sure when exactly the new format is going to be mandatory (CDDL is the only place where it says it is going to be mandatory in the new era).
The only thing I know for sure is that the recommendation is to upgrade all the toolings as soon as possible.

[rooooooooob]

So CML was never failing on the new networks? The thing with upgrading is that if it's in the next hardfork there are likely a bunch of other CDDL changes anyway so it would have been easier for us to just wait and make the changes then for us since we'd be doing CDDL-regeneration anyway. I guess for the redeemer case the builder part is now a bit easier to migrate over.