Skip to content
This repository was archived by the owner on May 22, 2025. It is now read-only.

DisputableVoting: Implement subgraph#178

Merged
facuspagnuolo merged 2 commits intomasterfrom
disputable_voting_subgraph
Aug 4, 2020
Merged

DisputableVoting: Implement subgraph#178
facuspagnuolo merged 2 commits intomasterfrom
disputable_voting_subgraph

Conversation

@facuspagnuolo
Copy link
Copy Markdown
Contributor

@facuspagnuolo facuspagnuolo commented Aug 3, 2020

You can try the subgraph here: aragon-dvoting-rinkeby-staging (there's a limit for the name's length)

@facuspagnuolo facuspagnuolo requested a review from 0xGabi August 3, 2020 15:14
@facuspagnuolo facuspagnuolo self-assigned this Aug 3, 2020
@codecov
Copy link
Copy Markdown

codecov Bot commented Aug 3, 2020

Codecov Report

Merging #178 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #178   +/-   ##
=======================================
  Coverage   20.43%   20.43%           
=======================================
  Files          69       69           
  Lines        1282     1282           
  Branches      234      234           
=======================================
  Hits          262      262           
  Misses       1020     1020           
Flag Coverage Δ
#unittests 20.43% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 87c0f7e...39bee4e. Read the comment docs.

@@ -0,0 +1,52 @@
{
"name": "@aragon/connect-voting-disputable",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Should we use the name @aragon/connect-disputable-voting? It seems to be a more common way to refer to it.

Comment thread packages/connect-voting-disputable/tsconfig.json Outdated
Comment thread packages/connect-voting-disputable/tsconfig.json Outdated
Copy link
Copy Markdown
Contributor

@0xGabi 0xGabi left a comment

Choose a reason for hiding this comment

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

Looking good!

vote.pauseDuration = voteData.value9
vote.quietEndingExtendedSeconds = voteData.value10
vote.quietEndingSnapshotSupport = castVoterState(voteData.value11)
vote.script = voteData.value12
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I wonder if the latest version of the contract is here: https://github.com/aragon/aragon-apps/blob/disputable_voting_deploy/apps/voting-disputable/contracts/DisputableVoting.sol#L85

I notice the voteData maybe have small differences.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yup, I double-checked and it is that one :)
We are also checking it in the tests

@facuspagnuolo facuspagnuolo merged commit a3cd199 into master Aug 4, 2020
@facuspagnuolo facuspagnuolo deleted the disputable_voting_subgraph branch August 4, 2020 19:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants