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

[Claim] Reject tx modals #2193

Merged
merged 5 commits into from
Jan 18, 2022
Merged

[Claim] Reject tx modals #2193

merged 5 commits into from
Jan 18, 2022

Conversation

W3stside
Copy link
Contributor

Summary

Merges into claim-reject-tx

Example 1:
image

Example 2:
image

To Test

  1. try claim direct (free claim)
  2. reject in wallet, see modal

approve

  1. try approve
  2. reject in wallet
  3. see modal

@W3stside W3stside requested review from a team January 18, 2022 17:18
@W3stside
Copy link
Contributor Author

that transparency is just from the screenshot

@github-actions
Copy link
Contributor

  • 🔭 GP Swap: Gnosis Protocol v2 Swap UI

Copy link
Contributor

@alfetopito alfetopito left a comment

Choose a reason for hiding this comment

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

👍

src/custom/hooks/useErrorMessageAndModal.tsx Show resolved Hide resolved
src/custom/hooks/useErrorMessageAndModal.tsx Outdated Show resolved Hide resolved
src/custom/pages/Claim/InvestmentFlow/InvestOption.tsx Outdated Show resolved Hide resolved
@W3stside W3stside merged commit adde661 into claim-reject-tx Jan 18, 2022
@W3stside W3stside deleted the claim-reject-tx-modals branch January 18, 2022 18:51
W3stside added a commit that referenced this pull request Jan 18, 2022
* useErrorMessage hook

* extend SwapCallbackError to be more generic

* add useErrorMessage to claim index & InvestOption

* add useErrorMessage to SwapMod

* [Claim] Reject tx modals (#2193)

* change hook name and add new hooks

* add new modal enum

* add hooks in app

* useMemo in useErrorMessageAndModal hook

* remove testing fn
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