Skip to content
This repository was archived by the owner on Jan 26, 2023. It is now read-only.

Conversation

@fmorency
Copy link
Contributor

Description

Remove Burn and Mint

Reference to minters were kept in LedgerStorage for future
reactivation

Depends on liftedinit/many-rs#66

Related Issue

Fixes #70

Testing

  • cargo test
  • run.sh

Breaking Change

  • Yes
  • No

Screenshots (if applicable)

Checklist:

  • I have read and followed the CONTRIBUTING guidelines for this project.
  • I have added or updated tests and they pass.
  • I have added or updated documentation and it is accurate.
  • I have noted any breaking changes in this module or downstream modules.

Reference to `minters` were kept in LedgerStorage for future
reactivation
@fmorency fmorency added the enhancement New feature or request label Apr 28, 2022
@fmorency fmorency self-assigned this Apr 28, 2022
@fmorency
Copy link
Contributor Author

CI will fail until liftedinit/many-rs#66 is merged and this PR updated.

@fmorency fmorency requested a review from hansl May 2, 2022 13:54
@fmorency fmorency marked this pull request as ready for review May 2, 2022 13:54
Copy link
Contributor

@hansl hansl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Nice cleanup.

@hansl hansl merged commit b36a0be into liftedinit:main May 2, 2022
@fmorency fmorency deleted the enh-70-remove-burn-mint branch June 15, 2022 12:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove Burn and Mint

2 participants