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

[Claim-Reject-Modals] Fixes empty modal #2204

Merged
merged 3 commits into from
Jan 18, 2022
Merged

Conversation

W3stside
Copy link
Contributor

Fixes empty modal as described by @elena-zh in #2195
Closes #2195

Test

  1. approve token
  2. reject
  3. (use acct with multiple investment options)

@W3stside W3stside requested review from a team January 18, 2022 20:22
@elena-zh
Copy link

Seems that the build preview link can't appear due conflicts 😐

@W3stside W3stside force-pushed the claim-reject-tx-modal-fix branch from 8aea13d to bea34ec Compare January 18, 2022 20:31
@W3stside
Copy link
Contributor Author

Seems that the build preview link can't appear due conflicts 😐

yeah sorry rebuilt it

@github-actions
Copy link
Contributor

  • 🔭 GP Swap: Gnosis Protocol v2 Swap UI

@W3stside
Copy link
Contributor Author

@gnosis/gp-qa @gnosis/gp-frontend gonna merge it, feel free to leave any comments after

@W3stside W3stside merged commit 7751865 into claim Jan 18, 2022
@elena-zh
Copy link

LGTM!
Thanks!

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.

2 participants