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

Fix free claims #2160

Merged
merged 4 commits into from
Jan 15, 2022
Merged

Fix free claims #2160

merged 4 commits into from
Jan 15, 2022

Conversation

alfetopito
Copy link
Contributor

Summary

Fixes issue preventing free claims from being listed in the details table

Screen Shot 2022-01-14 at 15 45 02

To Test

  1. Open the app on claim page, check any account that has both free and paid claims. There's a list a the bottom with some
  • Both free and paid claims are displayed in the table

Addresses

  • 0xfD40fcF9AD20C8fe4936E552833696496EA88E01
  • 0xC9A49b13A88Bd25E133AA1A709E5c31a2fc61591
  • 0x5239d4A1ff381f089D990Cd50B441cC9EFfE0ba3
  • 0x9470a102B9d6a3a00103537C2bC35ac0587Bd436
  • 0x2F144ff590C94871f118583E61fdDd06b98120e4
  • 0x42cE50b0387A081b51DAAE489DD4129CEc5a78f1
  • 0x6f14181e316377f683a9AE331B244717d38E619b
  • 0x05A1b71BF498d4A3930BA1f940E2D58C2150b399
  • 0x525Aa961cb6A422F325445F4529d58A5df3d19cc
  • 0x059E927e96079AffCC63583399641d1Da33e9Dd2
  • 0x1Ead91ed8Fd59B0742B17D25D224dC7D01CC17b1
  • 0xFA379cD1f24623eb282F007e0B7AC97eE5361657
  • 0x29d28C9b5a986809cb08504B54bfAEcFF79ad0BD
  • 0xEC45A4b65ddCe4da7e9D2F09Cddd31d1a38A4993
  • 0x904061135b2b0E87b1B40B641eC02ff07f1EF71B
  • 0x5B1bcdE24A407ce5032c1D2b59a83cf39F44604C
  • 0xB9EaC144Bf2b5331C722B087c323Fb6A5e8f04a6
  • 0xc6a4390A6f54a235B4Bec72236f84691A1903A25
  • 0xE8eAd5e33a24e228D86401f7dC84474d686BC40a
  • 0x07666F355a0E8882e92665aaFCd8241D4FDB0837

@alfetopito alfetopito self-assigned this Jan 14, 2022
@alfetopito alfetopito requested a review from a team January 14, 2022 23:51
@github-actions
Copy link
Contributor

  • 🔭 GP Swap: Gnosis Protocol v2 Swap UI

@W3stside W3stside merged commit d1cd9a6 into claim Jan 15, 2022
@W3stside W3stside deleted the fix-free-claims branch January 15, 2022 10:23
@anxolin
Copy link
Contributor

anxolin commented Jan 15, 2022

APPROVE. Small comments

@elena-zh
Copy link

LGTM!

@alfetopito alfetopito mentioned this pull request Jan 17, 2022
Merged
@alfetopito alfetopito mentioned this pull request Jan 25, 2022
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@anxolin anxolin anxolin left review comments

@W3stside W3stside W3stside approved these changes

Assignees

@alfetopito alfetopito

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@alfetopito @anxolin @elena-zh @W3stside