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

[suggestion] Genesis: change the Mint behaviour #4101

Open
6r1d opened this issue Nov 30, 2023 · 6 comments
Open

[suggestion] Genesis: change the Mint behaviour #4101

6r1d opened this issue Nov 30, 2023 · 6 comments
Assignees
@nxsaken
Labels
Enhancement New feature or request iroha2-dev The re-implementation of a BFT hyperledger in RUST

Comments

@6r1d
Copy link
Contributor

6r1d commented Nov 30, 2023 •
edited
Loading

Feature request

During our discussion with @Erigara in hyperledger-iroha/iroha-2-docs#440, we reached the conclusion Mint is grouping several separate behaviours.
If possible, altering its syntax or breaking it down into several similar ISI's could improve the UX.

Context:

Recently, Matías Salimbene asked us about the role of object field in Genesis Mint ISI.

Motivation

  • Improved UX
  • More intuitive command behaviour
  • Easier documentation

Who can help?

@mversic @Erigara
@6r1d 6r1d added Enhancement New feature or request iroha2-dev The re-implementation of a BFT hyperledger in RUST labels Nov 30, 2023
@Erigara
Copy link
Contributor

Erigara commented Nov 30, 2023

Might be also relevant for Burn and other operations.

@6r1d
Copy link
Contributor Author

6r1d commented Nov 30, 2023 •
edited
Loading

@mversic what do you think about changing Mint and Burn behaviours?

@Erigara
Copy link
Contributor

Erigara commented Nov 30, 2023

Find older issue with opposite proposal #3924

@6r1d
Copy link
Contributor Author

6r1d commented Nov 30, 2023

This proposal is opposite only in case the ISI is broken down.
It's not the opposite if the Mint / Burn structures are altered for better UX.

@Mingela
Copy link
Contributor

Mingela commented Dec 7, 2023

This might be partly related #4087 and considered together

@mversic
Copy link
Contributor

mversic commented Dec 7, 2023

@Erigara suggests to dissolve Mint instruction into several separate instructions. Mint<PublicKey> would then become AddPublicKey

@nxsaken nxsaken mentioned this issue Jun 25, 2024
Open
@nxsaken nxsaken self-assigned this Jun 25, 2024
@nxsaken nxsaken added this to Iroha Sep 4, 2024
@nxsaken nxsaken moved this to Backlog in Iroha Sep 4, 2024
@nxsaken nxsaken moved this from Backlog to Work in Progress in Iroha Sep 4, 2024
@nxsaken nxsaken moved this from Work in Progress to Backlog in Iroha Sep 4, 2024
@0x009922 0x009922 mentioned this issue Sep 10, 2024
Open
@mversic mversic moved this from Required for MVP to Backlog in Iroha Oct 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nxsaken nxsaken

Labels
Enhancement New feature or request iroha2-dev The re-implementation of a BFT hyperledger in RUST
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@6r1d @Mingela @nxsaken @Erigara @mversic