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

Governance API for deposits on a proposal returns null after voting period closes #2880

Closed
4 tasks
jmdfm opened this issue Nov 22, 2018 · 3 comments · Fixed by #3091
Closed
4 tasks

Governance API for deposits on a proposal returns null after voting period closes #2880

jmdfm opened this issue Nov 22, 2018 · 3 comments · Fixed by #3091

Comments

@jmdfm
Copy link
Contributor

jmdfm commented Nov 22, 2018

Summary of Bug

The Governance API for deposits on a proposal (/gov/proposals/:id/deposits) returns null instead of the deposits that were associated with the proposal after the voting period closes.

This is unexpected.

It would be expected that the voting period has no effect on the JSON payload returned by a call to /gov/proposals/:id/deposits, and that the call reliably returns the deposits cast against a proposal whenever it is called.

This may be related to #2879.

Steps to Reproduce

  1. Create a Proposal
  2. Make a Vote against a Proposal
  3. Call the Governance API to retrieve the deposits against a Proposal, see that you get a valid JSON payload as a response.
  4. Wait for the voting period to close.
  5. Make the same call you made in step 3, except this time you will receive null as a response.

For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@jmdfm
Copy link
Contributor Author

jmdfm commented Nov 22, 2018

Feel free to merge this with #2879 if its felt to be a duplicate of a core issue

@cwgoes
Copy link
Contributor

cwgoes commented Dec 11, 2018

This is expected.

The votes are deleted after the proposal has passed or failed, as they no longer need to be kept in state.

This endpoint will need to instead query past transactions and filter accordingly.

@alexanderbez alexanderbez self-assigned this Dec 11, 2018
@alexanderbez
Copy link
Contributor

I will be tackling this issue along with #2879.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants