Skip to content
This repository has been archived by the owner on Dec 13, 2019. It is now read-only.

[greenboard] Adds Deposit scenario #2135

Open
wants to merge 24 commits into
base: master
Choose a base branch
from
Open

Conversation

joelalejandro
Copy link
Contributor

@joelalejandro joelalejandro commented Aug 5, 2019

This PR implements the following scenarios:

A registered user should be able to make deposits with ETH.
A registered user should be able to make deposits with ERC20.

Contributes to #1852.

@joelalejandro joelalejandro added 🐢 Testing 📦 Greenboard code related to packages/greenboard labels Aug 5, 2019
@joelalejandro joelalejandro added this to the 👽 Grunty milestone Aug 5, 2019
@joelalejandro joelalejandro self-assigned this Aug 5, 2019
@joelalejandro joelalejandro changed the title [wip] [greenboard] Adds Deposit scenario [greenboard] Adds Deposit scenario Aug 7, 2019
@joelalejandro joelalejandro marked this pull request as ready for review August 7, 2019 05:31
@joelalejandro joelalejandro changed the base branch from joel/e2e-login to master August 7, 2019 19:52
@joelalejandro
Copy link
Contributor Author

Waiting on #2102 to be merged to prevent test selector conflicts.

@joelalejandro
Copy link
Contributor Author

joelalejandro commented Aug 14, 2019

@spersico can you pick up where I left off on this PR? It'd be great to add a second flow where the user attempts to deposit with an ERC20 token instead of ETH.

Don't worry too much about the conflicts; I'm 0x056ba3d73af34f0000 sure that those are related to selector names and legacy "deposit" references in Withdraw.

@spersico
Copy link
Collaborator

spersico commented Aug 14, 2019 via email

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
DO NOT MERGE 📦 Greenboard code related to packages/greenboard 🐢 Testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants