Skip to content
This repository has been archived by the owner on Jun 30, 2021. It is now read-only.

feat: OMG Network deposits from the admin panel #1222

Open
wants to merge 16 commits into
base: mederic-p/plasma-deposit
Choose a base branch
from

Conversation

kalouo
Copy link

@kalouo kalouo commented Nov 25, 2020

Issue/Task Number:

Closes #1194
Closes #1221

Screen Shot 2563-11-25 at 11 34 20

Overview

This PR wires up the deposit flow for the admin panel – along with a few miscellaneous enhancements.

Changes

Enables deposits from the admin panel

Miscellaneous:

  • Removes Metamask logic from the blockchain wallets page.
  • Adds OMG network configurations to blockchain settings page and orders all settings according to backend position.
  • Fixes "connected to Ethereum" message bug.

Implementation Details

Deposits:

  • Deletes HotWalletTransferChoser.js
  • Replaces it with a BlockchainActionSelector.tsx component refactored for extendability and reusability.
  • Refactors the blockchain wallet detail page (where BlockchainActionSelector is used) into TypeScript.
  • Refactors deposit action to send deposit amount as string type.
  • Corrects root chain/child chain identifier sent from client in root chain/child chain balance requests
  • Enable string amount parameter on /blockchain_wallet.deposit_to_childchain (with added test)

@kalouo kalouo changed the title Admin panel deposits feat: OMG Network deposits from the admin panel Nov 25, 2020
@kalouo kalouo changed the base branch from mederic-p/plasma-deposit to eth-blockchain November 25, 2020 03:20
@kalouo kalouo changed the base branch from eth-blockchain to mederic-p/plasma-deposit November 25, 2020 03:23
@kalouo kalouo marked this pull request as ready for review December 8, 2020 07:44
@kalouo kalouo requested a review from jarindr December 8, 2020 07:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants