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

ParaTimes: deposit and withdraw #969

Merged
merged 1 commit into from
Sep 9, 2022

Conversation

buberdds
Copy link
Contributor

@buberdds buberdds commented Aug 12, 2022

PR adds

  • deposit/withdraw
  • missing UI pieces

Next PRs will cover:
#978 unit tests, lock view for a 'foreign' account

Collapsible advance section
186187500-368d4cee-540a-458b-a269-699e3ffba879

Extended confirmation view
Screenshot 2022-08-25 at 18 32 58

Failed Transaction view
Screenshot 2022-08-25 at 18 41 55

src/app/lib/helpers.ts Outdated Show resolved Hide resolved
src/app/lib/transaction.ts Outdated Show resolved Hide resolved
src/app/lib/transaction.ts Outdated Show resolved Hide resolved
src/app/lib/transaction.ts Show resolved Hide resolved
src/app/lib/transaction.ts Outdated Show resolved Hide resolved
src/app/state/account/types.ts Show resolved Hide resolved
src/app/state/transaction/saga.ts Outdated Show resolved Hide resolved
src/app/state/paratimes/saga.ts Outdated Show resolved Hide resolved
src/app/state/transaction/saga.ts Outdated Show resolved Hide resolved
src/app/state/transaction/saga.ts Outdated Show resolved Hide resolved
@buberdds buberdds changed the title ParaTime deposit ParaTimes: deposit Aug 24, 2022
src/app/lib/transaction.ts Outdated Show resolved Hide resolved
src/app/lib/transaction.ts Outdated Show resolved Hide resolved
src/app/lib/transaction.ts Outdated Show resolved Hide resolved
src/config.ts Show resolved Hide resolved
src/app/lib/transaction.ts Outdated Show resolved Hide resolved
@buberdds buberdds changed the title ParaTimes: deposit ParaTimes: deposit and withdraw Aug 25, 2022
@buberdds
Copy link
Contributor Author

I will push more UI changes here

@buberdds buberdds force-pushed the buberdds/wizard-deposit branch 3 times, most recently from c4dfb89 to fe0feca Compare August 30, 2022 09:07
src/app/lib/transaction.ts Outdated Show resolved Hide resolved
src/app/lib/transaction.ts Outdated Show resolved Hide resolved
src/app/lib/transaction.ts Outdated Show resolved Hide resolved
src/app/lib/transaction.ts Outdated Show resolved Hide resolved
@buberdds buberdds force-pushed the buberdds/wizard-deposit branch 2 times, most recently from 91ebc52 to 7852af2 Compare August 31, 2022 12:20
src/app/pages/ParaTimesPage/index.tsx Outdated Show resolved Hide resolved
src/app/state/transaction/saga.ts Outdated Show resolved Hide resolved
src/app/state/account/saga.test.ts Outdated Show resolved Hide resolved
src/app/pages/ParaTimesPage/TransactionAmount/index.tsx Outdated Show resolved Hide resolved
src/app/state/transaction/saga.test.ts Outdated Show resolved Hide resolved
src/app/state/transaction/saga.ts Show resolved Hide resolved
src/app/state/transaction/saga.ts Outdated Show resolved Hide resolved
src/app/pages/ParaTimesPage/TransactionAmount/index.tsx Outdated Show resolved Hide resolved
Comment on lines 140 to 141
label="MAX"
onClick={() => setTransactionForm({ ...transactionForm, amount: formatter(balance) })}
onClick={() =>
Copy link
Member

Choose a reason for hiding this comment

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

MAX doesn't work with fees.

E.g. withdrawing MAX from cipher (after removing setAllowance) fails with Unknown gRPC error: withdraw: insufficient runtime balance

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I will address this one in a new PR.

src/app/pages/ParaTimesPage/index.tsx Outdated Show resolved Hide resolved
Comment on lines 42 to 46
Please enter the address of the withdrawing wallet on the
<strong>
Emerald
</strong>
(EVMc) ParaTime and then click "Next"
Copy link
Member

Choose a reason for hiding this comment

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

Text is misleading

  • When withdrawing from Emerald user should enter withdrawing private key and receiving address
    image

  • When withdrawing from Cipher user enters receiving address
    image

@buberdds buberdds merged commit eb8271d into oasisprotocol:paratimes Sep 9, 2022
@buberdds buberdds mentioned this pull request Sep 14, 2022
buberdds added a commit to buberdds/oasis-wallet-web that referenced this pull request Oct 25, 2022
buberdds added a commit to buberdds/oasis-wallet-web that referenced this pull request Oct 25, 2022
buberdds pushed a commit to buberdds/oasis-wallet-web that referenced this pull request Oct 26, 2022
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.

None yet

3 participants