-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
feat: Use sidekick server to handle CSV report download #3835
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Now that snapshot-labs/snapshot-sidekick#1 was merged can we move forward with getting this ready? |
I think this PR is pretty much done, the download button now return an error as expected, since sidekick is offline. Need to wait for the server deployment to start testing with real files |
We should be able to deploy this now on the new DO account @bonustrack created for you |
Since the server is empty, all download requests will end up with a "retry later" error, until cache is warmed up. Should we pre-warm all the biggest and recent proposals first ? |
@wa0x6e you mean this error? I think this is totally fine and we don't need to pre-warm any proposals. |
What happens if the user clicks download again while cache is ongoing? Will it interrupt anything on sidekick? |
I've been waiting a while and I still the error above when trying to download the votes here https://snapshot-nay16zdzi-snapshot.vercel.app/#/stgdao.eth/proposal/0x6b703b90d3cd1f82f7c176fc2e566a2bb79e8eb6618a568b52a4f29cb2f8d57b |
Just tried, it works. Generation may takes time, depending on number of votes. Count ~2s per 1000 votes |
No, duplicate are ignored |
I tried again here, it should take around 3 minutes but it's been taking longer than 10 |
Issue appears to come from the hub, rate limiting all the requests |
Please request another review once you've solved the rate limiting |
Rate limit will be solved by implementing the new API keys. For now, we just wait |
@samuveth Throttling issue has been resolved on sidekick. Can you try again ? |
Should be fixed via snapshot-labs/snapshot-sidekick#47 |
Co-authored-by: Sam <[email protected]>
Co-authored-by: Sam <[email protected]>
I assume it will only be fixed for proposals where the votes haven't been cached yet, because I still see the same count issue on the proposal above. |
Cache has been cleared. Should serve the correct version now |
Modal message UI has been updated |
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.
tACK
Can you fix the conflicts? |
Issues
Fixes #3756
Use https://github.com/snapshot-labs/snapshot-sidekick to handle the votes CSV report download
Changes
useReportDownload
composable to download the complete file from sidekick server, instead of building it from snapshot-hub requests.How to test
To-Do
Self-review checklist
yarn cypress:test:e2e
, and they have passedThis PR is ready, waiting to backend server to be online before merging