Skip to content
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

MonadBlockChain instance for Contract does not take TxOpts into account #129

Closed
florentc opened this issue Jun 16, 2022 · 2 comments
Closed
Labels
bug Something isn't working plutus v2 Should be considered in our efforts to integrate Plutus V2

Comments

@florentc
Copy link
Member

The MonadBlockChain instance for Plutus Contract does not take options TxOpts in transaction skeletons into account. Especially regarding balancing and ensuring every utxo has the minimum required amount of ada.

For example, take a skeleton that specifies a public key utxo should be spent to mint an NFT and put in at a script address, with the adjustUnbalTx option enabled. The instance for Contract generates a transaction with only one script output carrying only the NFT. Unlike the transaction generated by Direct, it misses a payback output for the public key and the 2 adas in the script output.

@florentc florentc added the bug Something isn't working label Jun 16, 2022
@VictorCMiraldo
Copy link
Contributor

This is definitely related to #78, which contains suggestions on fixing this

@carlhammann carlhammann added the plutus v2 Should be considered in our efforts to integrate Plutus V2 label Nov 1, 2022
@carlhammann carlhammann mentioned this issue Feb 1, 2023
4 tasks
@mmontin
Copy link
Collaborator

mmontin commented Feb 16, 2024

Closing this because this will be handled only when / if we decide to have an instance for the contract monad.

@mmontin mmontin closed this as not planned Won't fix, can't repro, duplicate, stale Feb 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working plutus v2 Should be considered in our efforts to integrate Plutus V2
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants