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

[feature] #3974: Add subcommands into iroha_client_cli to edit domain metadata #4175

Merged
merged 9 commits into from
Jan 15, 2024
Merged

[feature] #3974: Add subcommands into iroha_client_cli to edit domain metadata #4175

merged 9 commits into from
Jan 15, 2024

Conversation

VAmuzing
Copy link
Contributor

@VAmuzing VAmuzing commented Dec 25, 2023

Description

Fork of #4054 with a small update to from_str method

Linked issue

Closes #3974

Benefits

There won't be any confusions about why the same method for different structs implemented differently

Checklist

  • I've read CONTRIBUTING.md
  • I've used the standard signed-off commit format (or will squash just before merging)
  • All applicable CI checks pass (or I promised to make them pass later)
  • (optional) I've written unit tests for the code changes
  • I replied to all comments after code review, marking all implemented changes with thumbs up

Signed-off-by: Vladimir Pesterev <8786922+pesterev@users.noreply.github.com>
@github-actions github-actions bot added the iroha2-dev The re-implementation of a BFT hyperledger in RUST label Dec 25, 2023
Signed-off-by: VAmuzing <amuzik95@gmail.com>

Signed-off-by: VAmuzing <amuzik95@gmail.com>
@VAmuzing VAmuzing self-assigned this Dec 25, 2023
@Erigara Erigara self-assigned this Dec 26, 2023
client_cli/src/main.rs Outdated Show resolved Hide resolved
@0x009922 0x009922 self-assigned this Dec 26, 2023
client_cli/src/main.rs Show resolved Hide resolved
client_cli/src/main.rs Show resolved Hide resolved
Signed-off-by: VAmuzing <amuzik95@gmail.com>
…vor of ValueArg

Signed-off-by: VAmuzing <amuzik95@gmail.com>
Erigara
Erigara previously approved these changes Jan 15, 2024
0x009922
0x009922 previously approved these changes Jan 15, 2024
client_cli/src/main.rs Outdated Show resolved Hide resolved
Signed-off-by: VAmuzing <amuzik95@gmail.com>
@VAmuzing VAmuzing dismissed stale reviews from 0x009922 and Erigara via b9d46ee January 15, 2024 07:45
@VAmuzing VAmuzing merged commit 66072b4 into hyperledger-iroha:iroha2-dev Jan 15, 2024
6 of 9 checks passed
Asem-Abdelhady pushed a commit to Asem-Abdelhady/iroha that referenced this pull request Jan 22, 2024
…li to edit domain metadata (hyperledger-iroha#4175)

Signed-off-by: Vladimir Pesterev <8786922+pesterev@users.noreply.github.com>
Signed-off-by: VAmuzing <amuzik95@gmail.com>
Co-authored-by: Vladimir Pesterev <8786922+pesterev@users.noreply.github.com>
Signed-off-by: Asem-Abdelhady <asemshawkey@gmail.com>
Asem-Abdelhady pushed a commit to Asem-Abdelhady/iroha that referenced this pull request Feb 9, 2024
…li to edit domain metadata (hyperledger-iroha#4175)

Signed-off-by: Vladimir Pesterev <8786922+pesterev@users.noreply.github.com>
Signed-off-by: VAmuzing <amuzik95@gmail.com>
Co-authored-by: Vladimir Pesterev <8786922+pesterev@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
iroha2-dev The re-implementation of a BFT hyperledger in RUST
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants