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

Claim hooks 2 - ready to fetch #2043

Merged
merged 10 commits into from
Jan 3, 2022
Merged

Claim hooks 2 - ready to fetch #2043

merged 10 commits into from
Jan 3, 2022

Conversation

alfetopito
Copy link
Contributor

@alfetopito alfetopito commented Dec 24, 2021

Summary

Adapted fetch functions to expected document format coming from future claims repo

  • Modded useUserClaims

  • Using fake data but at a different stage

  • Converting string types to enum types (Airdrop -> 0)

  • Renamed button Claim vCOW to vCOW

  • Made vCOW button always visible

    To Test

  1. Connect wallet on rinkeby
  2. Click on vCOW button
  3. Click on claim
  4. Execute tx
  • Claim will be performed and claimable amount will be reduced

Note: Each claim can be done only once, so if you can't claim, try using a different claim index in this file
Screenshot from 2021-12-24 17-08-53

Background

Optional: Give background information for changes you've made, that might be difficult to explain via comments

@alfetopito alfetopito self-assigned this Dec 24, 2021
@alfetopito alfetopito requested a review from a team December 24, 2021 20:10
Copy link
Contributor

@fedgiac fedgiac left a comment

Choose a reason for hiding this comment

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

I tried the test plan and after I connected my Metamask it worked, here is the transaction. ✔️

I can't say much about how React-specific things are used, but everything contract related looks good to me.

@github-actions
Copy link
Contributor

github-actions bot commented Jan 3, 2022

  • 🔭 GP Swap: Gnosis Protocol v2 Swap UI

@alfetopito
Copy link
Contributor Author

FYI, I rebased with latest from claim branch which includes Michel's components, thus the claim action no longer works.
Since Federico tested and it's working, I'll merge as is.
Feel free to add comments afterwards and I'll address them in follow up PRs.

@alfetopito alfetopito merged commit 8d53e6b into claim Jan 3, 2022
@alfetopito alfetopito deleted the claim-hooks-2 branch January 3, 2022 22:00
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