-
Notifications
You must be signed in to change notification settings - Fork 16
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
feat(iota)!: unordered optional IotaClientCommands::NewAddress
arguments
#4647
feat(iota)!: unordered optional IotaClientCommands::NewAddress
arguments
#4647
Conversation
|
This pull request has been deployed to Vercel. Latest commit: 148e666 |
This pull request has been deployed to Vercel. Latest commit: 148e666 ✅ Preview: https://wallet-dashboard-832fpi7t6-iota1.vercel.app |
This pull request has been deployed to Vercel. Latest commit: 148e666 ✅ Preview: https://rebased-explorer-b5mypwdsb-iota1.vercel.app |
✅ Vercel Preview Deployment is ready! |
IotaClientCommands::NewAddress
argumentsIotaClientCommands::NewAddress
arguments
Co-authored-by: Thoralf-M <46689931+Thoralf-M@users.noreply.github.com>
This pull request has been deployed to Vercel. Latest commit: 8e7cc10 |
This pull request has been deployed to Vercel. Latest commit: 8e7cc10 ✅ Preview: https://wallet-dashboard-fu4p39d9x-iota1.vercel.app |
This pull request has been deployed to Vercel. Latest commit: 8e7cc10 ✅ Preview: https://rebased-explorer-fz2h5qzmj-iota1.vercel.app |
✅ Vercel Preview Deployment is ready! |
IotaClientCommands::NewAddress
argumentsIotaClientCommands::NewAddress
arguments
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM :)
✅ Vercel Preview Deployment is ready! |
This pull request has been deployed to Vercel. Latest commit: ce9feae |
This pull request has been deployed to Vercel. Latest commit: ce9feae ✅ Preview: https://rebased-explorer-makt4qlpl-iota1.vercel.app |
This pull request has been deployed to Vercel. Latest commit: ce9feae ✅ Preview: https://wallet-dashboard-71yhvljn6-iota1.vercel.app |
This pull request has been deployed to Vercel. Latest commit: be3d954 |
This pull request has been deployed to Vercel. Latest commit: be3d954 ✅ Preview: https://rebased-explorer-l59g9arr1-iota1.vercel.app |
This pull request has been deployed to Vercel. Latest commit: be3d954 ✅ Preview: https://wallet-dashboard-1kvmwdsoq-iota1.vercel.app |
✅ Vercel Preview Deployment is ready! |
✅ Vercel Preview Deployment is ready! |
This pull request has been deployed to Vercel. Latest commit: df12c36 ✅ Preview: https://rebased-explorer-ktmqn1g69-iota1.vercel.app |
This pull request has been deployed to Vercel. Latest commit: df12c36 |
This pull request has been deployed to Vercel. Latest commit: df12c36 ✅ Preview: https://wallet-dashboard-1n62pr564-iota1.vercel.app |
…ments (#4647) * cli: unordered optional `IotaClientCommands::NewAddress` arguments * optional key_scheme * docs * command doc * Update docs/content/developer/dev-cheat-sheet.mdx Co-authored-by: Thoralf-M <46689931+Thoralf-M@users.noreply.github.com> --------- Co-authored-by: Thoralf-M <46689931+Thoralf-M@users.noreply.github.com>
Change the arguments of
IotaClientCommands::NewAddress
to be unordered and with flags. This is more consistent with the other client commands as this is the only one with ordered arguments. This is also a better UX as we currently have to provide all arguments if we only want to provide the last one. We can now provide only what is wanted.Also made the key-scheme optional defaulting to Ed25519.