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.

Sign up for GitHub

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

Jump to bottom

dApp: refactor '23 #154

Closed
7 of 29 tasks
moconnell opened this issue Dec 6, 2023 · 0 comments · Fixed by #155
Closed
7 of 29 tasks

dApp: refactor '23 #154

moconnell opened this issue Dec 6, 2023 · 0 comments · Fixed by #155
Assignees
@moconnell
Labels
frontend refactor tech-debt

Comments

@moconnell
Copy link
Collaborator

moconnell commented Dec 6, 2023 •
edited
Loading

  • Migrate to TypeScript
    • rename files to .ts(x?)
    • fix type errors
  • remove Web3Context
    • remove ethers & web3 refs
    • use viem & wagmi instead
  • update invoices page
    • update @tanstack/react-table
    • use @tanstack/react-query
    • urql & fake-tag -> graphql-request
  • update invoice creation
    • remove CreateContext
    • Use react-hook-form & Dungeon Master form components
    • Use FieldArray for milestones instead of 2 steps
    • Hooks for reads & contract interactions
    • Move forms to their own
  • Transaction handling in OverlayContext
  • use Chakra useToast hook for user notifications
  • Use ChakraNextLink
  • Use Chakra mobile query handling
  • Use IPFS pin hook
  • ipfs-http-client to multiformats
  • use Piñata?
  • Consolidate public & assets (Repeat usage? Can all be public?)
  • theme
    • Application Button
    • Heading & Text
  • Remove 3box -> useEnsName + useEnsAvatar
  • add basic component tests
@moconnell moconnell self-assigned this Dec 6, 2023
@moconnell moconnell linked a pull request Dec 6, 2023 that will close this issue
dApp Refactor '23 #155
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@moconnell moconnell

Labels
frontend refactor tech-debt
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

dApp Refactor '23 SmartInvoiceXYZ/smart-invoice
1 participant
@moconnell