Skip to content

Fund (DAO) Creation UI#69

Open
BDANG wants to merge 55 commits into
developfrom
fund-creation
Open

Fund (DAO) Creation UI#69
BDANG wants to merge 55 commits into
developfrom
fund-creation

Conversation

@BDANG
Copy link
Copy Markdown
Contributor

@BDANG BDANG commented Jun 8, 2022

This branch includes Eric's multi-transaction broadcaster so we should merge that first.


  • Implemented a UI for creating funds. THe multi "page" modal collects user-input
  • Parent component CreateFund.tsx manages state and provides them to views in src/components/create-fund-views/*.tsx. This should enable more flexible & compartmentalized interfaces for fund-creation
  • The modal correctly invokes the dao-creation transactions, however very few inputs are provided to the transactions

Follow-up issues will improve the UI/UX and on-board user-inputs to the solana transaction(s)

Copy link
Copy Markdown
Contributor

@tjkyner tjkyner left a comment

Choose a reason for hiding this comment

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

Awesome work! Excited to see how this continues to progress. No rush on getting this merged in since we don't actually want that functionality out there yet so feel free to keep iterating on it for now.

Comment thread src/App.tsx Outdated
Comment thread src/components/CreateFund.tsx
Comment thread src/components/CreateFund.tsx
Comment thread src/components/SubNavbar.tsx
Comment thread src/components/SubNavbar.tsx Outdated
Comment thread src/components/create-fund-views/FundConfig.tsx Outdated
Comment thread src/components/create-fund-views/FundName.tsx
Comment thread src/models/constants.ts
import {PublicKey} from "@solana/web3.js";

// @TODO: Move this to the client side env
export const USDC_LOCALNET = new PublicKey("8GM1dEZR6sL9Kut9GpSuPC5FTVCANeTNJrq31bepuDXU")
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We'll need this be in it's own .env file since it'll be different for each dev

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Similar comment above, change should happen in Eric's branch and then I'll merge it into here

@tjkyner tjkyner linked an issue Jun 9, 2022 that may be closed by this pull request
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

Successfully merging this pull request may close these issues.

Flow for creating an investment fund

2 participants