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

feat: DAI Fast Withdrawals #190

Draft
wants to merge 212 commits into
base: dev
Choose a base branch
from
Draft

Conversation

nulven
Copy link

@nulven nulven commented May 16, 2022

Description of the Changes

Please add a detailed description of the change, whether it's an enhancement or a bugfix.
If the PR is related to an open issue please link to it.

Checklist

  • Changes have been done against master branch, and PR does not conflict
  • New unit / functional tests have been added (whenever applicable)
  • Test are passing in local environment
  • Docs have been updated
  • PR title is follow the Conventional Commits convention: <type>[optional scope]: <description>, e.g: fix: prevent racing of requests

This change is Reviewable

dan-ziv and others added 30 commits November 30, 2021 23:44
semantic-release-bot and others added 22 commits May 3, 2022 21:13
 ## [1.4.0-dev.1](starknet-io/starkgate-frontend@v1.3.1-dev.5...v1.4.0-dev.1) (2022-05-03)

### 🧩 Features

* add asset to SN wallet after a successful deposit ([starknet-io#171](starknet-io#171)) ([30d8ec3](starknet-io@30d8ec3))
 ## [1.4.0](starknet-io/starkgate-frontend@v1.3.0...v1.4.0) (2022-05-04)

### 🔧 Fixes

* cancel toFixed when format balances ([starknet-io#172](starknet-io#172)) ([2906e3e](starknet-io@2906e3e))
* connection modal overrides gsw selection modal ([starknet-io#175](starknet-io#175)) ([73b1f21](starknet-io@73b1f21))
* temporary disable max button ([starknet-io#173](starknet-io#173)) ([1d3038d](starknet-io@1d3038d))
* update app state on accountsChanged event ([starknet-io#176](starknet-io#176)) ([3161cc9](starknet-io@3161cc9))
* update text for the attempt to deposit more than the allowable amount ([starknet-io#174](starknet-io#174)) ([d8f3da8](starknet-io@d8f3da8))

### 🧩 Features

* add asset to SN wallet after a successful deposit ([starknet-io#171](starknet-io#171)) ([30d8ec3](starknet-io@30d8ec3))
 ### [1.4.1-dev.1](starknet-io/starkgate-frontend@v1.4.0...v1.4.1-dev.1) (2022-05-08)

### 🔧 Fixes

* CompleteTransferToL1 toast remains on display after tx is done ([starknet-io#179](starknet-io#179)) ([219b860](starknet-io@219b860))
 ### [1.4.1-dev.2](starknet-io/starkgate-frontend@v1.4.1-dev.1...v1.4.1-dev.2) (2022-05-08)

### 🔧 Fixes

* **Header:** testnet title not displayed ([4169584](starknet-io@4169584))
 ### [1.4.1](starknet-io/starkgate-frontend@v1.4.0...v1.4.1) (2022-05-08)

### 🔧 Fixes

* CompleteTransferToL1 toast remains on display after tx is done ([starknet-io#179](starknet-io#179)) ([219b860](starknet-io@219b860))
* **Header:** testnet title not displayed ([4169584](starknet-io@4169584))
 ### [1.4.2-dev.1](starknet-io/starkgate-frontend@v1.4.1...v1.4.2-dev.1) (2022-05-10)

### 💉 Improvements

* export convention ([starknet-io#180](starknet-io#180)) ([b5585ad](starknet-io@b5585ad))
 ## [1.5.0-dev.1](starknet-io/starkgate-frontend@v1.4.2-dev.2...v1.5.0-dev.1) (2022-05-11)

### 🧩 Features

* MultiChoiceMenu component ([starknet-io#181](starknet-io#181)) ([bbf19ba](starknet-io@bbf19ba))

### 🔧 Fixes

* change transfer tabs names ([starknet-io#182](starknet-io#182)) ([e795f1c](starknet-io@e795f1c))
@dan-ziv dan-ziv self-requested a review May 18, 2022 06:36
src/hooks/useTransferToL1.js Outdated Show resolved Hide resolved
@@ -103,6 +116,172 @@ export const useTransferToL1 = () => {
);
};

export const useFastTransferToL1 = () => {

Choose a reason for hiding this comment

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

What will happen if frontend is reloaded after fast transfer is initiated? Will it resume the withdrawal process?

@dan-ziv dan-ziv changed the title DAI Fast Withdrawals feat: DAI Fast Withdrawals May 19, 2022
@dan-ziv dan-ziv marked this pull request as draft August 29, 2022 08:53
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.

7 participants