Add poap-unlock strategy [poap-unlock]

[paulperegud]

Adds POAP-unlock voting strategy.

User has one vote if they have any POAPs and zero otherwise. If eventIds are specified, than only POAPs of particular event ids are taken into account.

[ChaituVR]

Hi @paulperegud I see this is same as poap strategy except the check for minimum, you can use our voting validation feature for this, https://docs.snapshot.org/user-guides/strategies/validation-strategies#voting-validation-in-space-settings

[ChaituVR]

Closing this PR for now, feel free to reopen if needed :)

[vpabundance]

@ChaituVR I've been the one looking to set this up. Under the link provided, this appears to be for Proposal Validation. Can I ask how this would be applied to a strategy where users with 1, or multiple poaps have the same voting power?

[ChaituVR]

Ah sorry, this one https://docs.snapshot.org/user-guides/strategies/validation-strategies#validation-strategy-example-basic

an I ask how this would be applied to a strategy where users with 1, or multiple poaps have the same voting power?
You can use it the way you want. after adding voting validation, use ticket strategy to get VP. so it will give one VP

[vpabundance]

Apologies for my lack of understanding here. "The Basic validation strategy allows you to use existing Voting Strategies configured for your space or define a custom setup to determine if a user is eligible to create a proposal or cast a vote. "

When I go to set this up, all I am seeing is the ability to add which poap events allow a user to be eligible to vote. When a user has both poaps in their wallet, they have x2 voting power.

Not seeing where we can limit that scenario to 1 under the link you provided.

[ChaituVR]

So voting validations are just for yes or no, if validation return true, then using space voting strategies, setup a ticket strategy like here https://docs.snapshot.org/user-guides/spaces/settings#voting-strategies it will give 1 voting power to everyone who is validated with voting validation

[vpabundance]

Wouldn't this require then the use of passport?

[vpabundance]

Gitcoin is one of the 10 projects in this vote, and while we support them, it's not going to reflect well if voting requires the use of one of their products as this would reduce voters outside of their community.

[paulperegud]

@ChaituVR This PR is something different from ticket option. Yes, both give user just 1 vote, but ticket option requires user to have gitcoin passport. Please reopen (I lack permissions to do that).

[ChaituVR]

Hey guys we don't require gitcoin validation anymore, you can use basic validation too. in basic validation, you can use poap strategy and set the minimum score to your desired number of tokens required to vote, and the ticket strategy to give 1 voting power

Sorry those docs are outdated. @zuz could you update that plz :) #1209 (comment)

[vpabundance]

Having trouble getting what you are suggesting to work. Here is what I have under: valdation>basic:

{
"symbol": "POAP",
"eventIds": [
"128253",
"128044"
]
}

And then additionally ticket with voting power of 1.

This still returns a voting power of 2 should a wallet have 2 poaps.

Are you able to look at the setup on https://snapshot.org/#/octantapp.eth/settings and see what I am missing here?
Thank you for any insight!

[zzuziak]

Hey @vpabundance, the setup seems right, and the ticket strategy always gives only 1 VP. Did you check it on a new proposal after updating the settings? Keep in mind that the update in settings affects only new strategies.

[vpabundance]

Thank you so much to everyone here who helped, I successfully created and tested the voting. Cheers!