-
Notifications
You must be signed in to change notification settings - Fork 25
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
Integrate Snapshot's daoURI with the attestation framework #205
Comments
It seems that the 'membersURI' from the Snapshot eip4824 API just contains space members, which are not voters. |
What we need to do here next is clearly describe what it means to publish daoURI data through an attestation, i.e. since Snapshot will be publishing daoURI data about DAOs through an attestation. This issue is already being addressed in part through #99 and #100 for Optimism, i.e. because the Optimism Foundation needs to publish daoURI data about the Optimism Collective through an attestation. |
Created a PR on Snapshot to include voters in their membersURI Conveyed this to their team, waiting for their review and response on this |
Fabien's review,
Based on this, I've added limit, pagination, and uniqueness checks for members. |
Pushed changes, snapshot-labs/snapshot-hub#812 |
Just taking a long time, ball is still in their court to review Rashmi's PR. |
This was combined with #239 , will close for now. |
Display the DAO URI of snapshot spaces, published by snapshot https://hub.snapshot.org/api/eip4824/<space_name>
on our frontend with disclaimer that the data provided is by Snapshot and NOT by DAO themselves
The text was updated successfully, but these errors were encountered: