Skip to content
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

Adds APIs to support gathering of minimal export of grants contributor addresses, with no other information exported. #7655

Merged
merged 10 commits into from
Oct 15, 2020

Conversation

owocki
Copy link
Contributor

@owocki owocki commented Oct 12, 2020

Description

Adds APIs to support gathering of minimal export of grants contributor addresses, with no other information exported.

This is useful so that if projects ever want to send a POAP or any other NFT to celebrate their conributions to grants, to these users, they can now do so.

There are a few exports available

  1. users can get their entire earnings history at /settings/account
  2. users who are team members of a grant can now export the contribution history to that grant
  3. users can now get a list of all contirbutor addresses to a specific round
Refers/Fixes

https://twitter.com/owocki/status/1314593279988568066

Testing

Testedl locally

end = timezone.datetime(2020, 7, 3)
if round_id == 7:
start = timezone.datetime(2020, 9, 15)
end = timezone.datetime(2020, 10, 3)
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i suppose in future rounds we could lookup in the GrantCLR table, but the fact that the 'round_num' field of that table isntreally a number makes me feel like its not perfect..

@owocki owocki changed the base branch from kevin/sync_tx_info to stable October 15, 2020 19:17
@owocki owocki merged commit 329fb81 into stable Oct 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant