-
Notifications
You must be signed in to change notification settings - Fork 213
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Move Redeemer
type to Write.Tx.Redeemers
module
#4047
Conversation
c298a89
to
e59f9f4
Compare
e59f9f4
to
53b37d4
Compare
@@ -223,6 +242,50 @@ assignScriptRedeemers pparams timeTranslation utxo redeemers tx = | |||
, Just l <- [Alonzo.language script] | |||
] | |||
|
|||
-- | |||
-- The 'Redeemer' type | |||
-- TODO: Move back to the wallet and/or retire |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👆 We should (probably) work with the redeemers a part of the partial tx CBOR instead. In that case we'd need to add logic for updating the tx without invalidating the existing ix-based pointers.
) where | ||
|
||
import Prelude | ||
|
||
import Cardano.Api | ||
( StakeAddress, serialiseToBech32 ) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should have the cardano-api
imports qualified as Cardano
instead.
Primitive.Types.Redeemers
intoWrite.Tx.Redeemers
, while avoiding dependency onShelley.Compatibility
Comments
Issue Number
ADP-3081