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.

Sign up for GitHub

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

Jump to bottom

Create a newtype for Value to represented minted/burned tokens #6445

Closed
zliu41 opened this issue Sep 4, 2024 · 0 comments · Fixed by #6609
Closed

Create a newtype for Value to represented minted/burned tokens #6445

zliu41 opened this issue Sep 4, 2024 · 0 comments · Fixed by #6609
Assignees
@Unisay
Labels
Internal status: triaged tech debt

Comments

@zliu41
Copy link
Member

zliu41 commented Sep 4, 2024 •
edited
Loading

It has been suggested in #5781 (comment) that among all Value fields in ScriptContext, only txInfoMint could contain negative quantities.

  • Check with the ledger team whether this is the case
  • If so, it could make sense to make a newtype wrapper to represent txInfoMint.
@github-actions github-actions bot added the status: needs triage GH issues that requires triage label Sep 4, 2024
@zliu41 zliu41 mentioned this issue Sep 4, 2024
Merged
@ramsay-t ramsay-t added Feature status: triaged Internal and removed status: needs triage GH issues that requires triage labels Sep 10, 2024
@Unisay Unisay self-assigned this Oct 15, 2024
@Unisay Unisay mentioned this issue Oct 29, 2024
Merged
11 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Unisay Unisay

Labels
Internal status: triaged tech debt
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

MintValue IntersectMBO/plutus
4 participants
@Unisay @ramsay-t @zliu41 @effectfully