-
Notifications
You must be signed in to change notification settings - Fork 7
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
README update #80
README update #80
Conversation
CJ-Rose
commented
Apr 1, 2024
•
edited
Loading
edited
- remove unused, add new tech and its role to stack section
- edit copy, add links to IPFS and ERC-1155 in 'Data Models' section
- re-organize 'Getting Started' section to include instructions to run locally
- update 'Prerequisites' for Next and Sepolia network config
- add introductory info and docs links for all tech to 'Helpful References'
✅ Deploy Preview for voicedeck ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
I've added intro descriptions of all the tools used in the repo, thinking it would be helpful for incoming contributors (including less-technical) to get an overview of all the moving parts. Might be too much, idk, what does everyone think? 'Server Design' section needs to be fleshed out more.. currently only Would 'Data Models' section be a good place to add a few more definitions of concepts, like 'Impact Funding', 'BC ratio'? or better to leave those to the FAQ page? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lets strip this back to get a dev up and running so they can read the code for themselves. I think there's a lot of stuff that helpful in here though, thank you!
Thanks for the update! Feel free to remove whole |
✅ Deploy Preview for vd-dev ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |