Skip to content

feat: add possibility to enter ICP and e8s as amount of cycles to fab…#2154

Merged
viviveevee merged 6 commits intomasterfrom
SDK-426-icp-to-cycles-conversion
Apr 26, 2022
Merged

feat: add possibility to enter ICP and e8s as amount of cycles to fab…#2154
viviveevee merged 6 commits intomasterfrom
SDK-426-icp-to-cycles-conversion

Conversation

@viviveevee
Copy link
Copy Markdown
Contributor

@viviveevee viviveevee commented Apr 22, 2022

…ricate

dfx ledger fabricate-cycles can now accept amounts in ICP or e8s, making it a drop-in replacement for dfx ledger top-up. It fetches the current exchange rate from the real IC.

Description

extension of #2135

Fixes: SDK-426

How Has This Been Tested?

Manually. Don't know how to mock IC responses within bats.
image

Checklist:

  • The title of this PR complies with Conventional Commits.
  • I have edited the CHANGELOG accordingly.
  • I have made corresponding changes to the documentation.

@viviveevee viviveevee requested a review from a team as a code owner April 22, 2022 12:33
Comment thread src/dfx/src/util/currency_conversion.rs Outdated
Co-authored-by: Eric Swanson <64809312+ericswanson-dfinity@users.noreply.github.com>
@ghost ghost added the automerge-squash label Apr 26, 2022
@viviveevee viviveevee enabled auto-merge (squash) April 26, 2022 16:39
@viviveevee viviveevee merged commit d732c42 into master Apr 26, 2022
@viviveevee viviveevee deleted the SDK-426-icp-to-cycles-conversion branch April 26, 2022 16:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant