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

To Do #4

Open
1 of 12 tasks
RohanNero opened this issue Nov 28, 2023 · 0 comments
Open
1 of 12 tasks

To Do #4

RohanNero opened this issue Nov 28, 2023 · 0 comments

Comments

@RohanNero
Copy link
Owner

RohanNero commented Nov 28, 2023

To Do:

  • Style pages
  • Implement dynamic chain functionality
  • Ensure output for every page
  • Write additional smart contract tests
  • Check validity of merkle root when creating event. (Construct the tree using other input variables)
  • Update API calls to use Viem instead when sensible (leaves, proof, tree)
  • Create helper file containing list of deployed Claim contracts
  • Revise logic pertraining to nfts/totalSupply variables, maybe allow this to be an optional variable that uses totalSupply() with ERC721Enumerable contracts
  • Update createEvent to have approve logic for the reward token if not using native ETH.
  • Update createClaim to calculate heldUntil on the backend so the user doesn't have to
  • Allow to to be empty on createClaim if user wants to send funds to the same address they held with
  • Handle client side error from invalid .call params in useAccountBalance hook
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

No branches or pull requests

1 participant