kalidao / kali-contracts Goto Github PK

I got this project fresh and ran into errors doing npx hardhat compile. These are the changes I had to make:

in contracts/multi-sig/ClubSig.sol:
change import '../tokens/ERC721/ERC721initializable.sol'; to import '../tokens/erc721/ERC721initializable.sol'; (casing on directory)
add import '../utils/Multicall.sol';

in contracts/tokens/erc721/KaliNFT.sol:
add import '../../../utils/Multicall.sol';

Compilation worked after that.

Then tests failed. Steps to correct the tests and get them working:

In KaliDAO.test.js:
There is something called FixedERC20 which is not a contract in the repo. When changing it to KaliERC20 it appears that this was not meant to be the correct contract as FixedERC20 seems to include a constructor. I was not able to get past this.

Maybe the two main requirements from smart contract perspective are to limit the manager grant to a certain ceiling of tokens and an expiration deadline for the minting powers. Maybe similar parameters for burning.

by Ivelin

extension that burns and mints a non-transferrable token

it's cool

Didn't know this but I set up a Test DAO with token - bigupyourself - https://rinkeby.etherscan.io/token/0x9dd998bb51b2971fefd1724029883ea5ff99c63d

When I try to import token to MM, I get an error stating 'Symbol must be 11 characters or fewer'. Perhaps we should limit the Token Name field on UI to 11 characters to make sure DAOs can actually import tokens into MM.