[erc721-with-landtype-multiplier] Add strategy erc721-with-landtype-multiplier

snapshot-labs / snapshot-strategies

Snapshot voting strategies

https://docs.snapshot.org/strategies

MIT License

223 stars 773 forks source link

[erc721-with-landtype-multiplier] Add strategy erc721-with-landtype-multiplier #1529

Open monish-nagre opened 3 months ago

monish-nagre commented 3 months ago

Fixes # .

Changes proposed in this pull request:

This strategy returns the balances of the voters for a specific ERC721 NFT with an arbitrary multiplier based on the type of land they own. Types Of Land : Mega contributes 25000 VP Large contributes 10000 VP Medium contributes 4000 VP Unit contributes 2000 VP

monish-nagre commented 3 months ago

Actually , i'm confuse that my request is accept or not , because the testcase are run as i expected voting power. the error is for example.json address range like 3 and etc , i think that doesn't effect my strategy. Please conform me. My request is ready to merge / having issue ?

monish-nagre commented 3 months ago

Please @ChaituVR , check the updated code.

ChaituVR commented 3 months ago

It should pass all test cases @monish-nagre

 call revert exception [ See: https://links.ethers.org/v5-errors-CALL_EXCEPTION ] (method="balanceOf(address)", data="0x", errorArgs=null, errorName=null, errorSignature=null, reason=null, code=CALL_EXCEPTION, version=abi/5.6.4)