-
Notifications
You must be signed in to change notification settings - Fork 36
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
accountsWithUnclaimedRewards_contains should not contain CR beneficiaries for proposals that are ExpiredInQueue #455
Comments
…e proposals We were showing the rewards as redeemable if the proposal had more pass votes than fail. There is still a small bug here, the proposal will still appear in the Redeem menu though it wont show any rewards as actually available. This has to be fixed by daostack/subgraph#455 Also fix little UI bug on voting buttons with overlapping text
Hi, can you please provide a query which shows an example of the problem? |
{ fragment ProposalFields on Proposal { |
…e proposals (#1334) We were showing the rewards as redeemable if the proposal had more pass votes than fail. There is still a small bug here, the proposal will still appear in the Redeem menu though it wont show any rewards as actually available. This has to be fixed by daostack/subgraph#455 Also fix little UI bug on voting buttons with overlapping text
It still needs to return GP rewards so stakers on expired proposals can get their GEN back |
* Fix daostack#455 * Fix
* v37-4 * Update migration * Update mappings.json * Update buffidao.json * Sponsor daos xdai * Fix daostack#455 * Fix competition test * Update Competition.spec.ts
* Add architecture image of Subgraph (#328) * Add files via upload * Create images * Delete images * Create create * Add files via upload * Delete 190910_DS_SUBGAPH_DIAGRAM_V01-2.jpg * Delete create * Update README.md * Add files via upload * Update README.md * Update README.md * Delete 190910_DS_SUBGAPH_DIAGRAM_V01-2.jpg * Index v33 on rinkeby (#401) * Use migration v34 (#402) * Use migration v34 * Fix * lint * Fix * Deprecate UGenericScheme * Index mainnet and Rinkeby v34 * Fix contracts info generator (#404) * genesis alpha workaround (#405) * WIP: Support Competition Scheme (#408) * Support competition scheme * Fix test (wip) * Competition test get proposal * Add competition redeem * Update mapping.ts * Support descriptionHash * fix * Bump version * fix ipfs test * Fix 407 (#410) * Fix #407 * Fix #409 * lint * Update tests * Update graph node to v0.16.1 (#411) Fix #395 * fix #412 (#414) * fix #412 * Bump version to 36 * Fix 415 (#416) * Index only v16 and v36 * fix * Index v33 * Fix and bump version * Return v19 * Index v32 (#418) * Fix #417 * Bump version * Fix #406 (#413) * Fix #406 * use fixJsonQuotes instead of replace * add package-lock.json * updsate package-lock.json * bump v to 0.0.36-3 * Fix #420 (#421) * Fix #420 * lint * Fix * Add standalone contracts to contractsInfo * Fix contracts info ganache * Fix bug * Fix tests * Fix (#422) * Fix * Bump version * Create nectardao.json (#423) * Create nectardao.json * Delete necdao.json * New nec (#424) * New nec * Create dummydao.json * New necdao (and dummy rinkeby dao) (#426) * Update nectardao.json (#428) * New Nectar (#431) * Fix reptutation typo in Competition (#433) * Index mainnet v31, v32, v33 (#432) * Fix stupid mistake in ContributionRewardExt (#434) * Fix stupid mistake in ContributionRewardExt * Bump version * Support migration v36-2 (rinkeby v36 contracts) (#435) * Support migration v36-2 (rinkeby v36 contracts) * Update docker ganche * Remove mistake mappings entry (#436) * graphnode v0.17.0 (#437) * graphnode v0.17.0 * fix daotoken test * support xdai (#443) * support xdai * update docker migration * fix sanity test * tests * index dxdao ens generic schemes (#444) * Competition suggestion extras (#440) * Add suggestion fields Fix #438 #439 * lint * fix test * bump version * Bump migration version * return migration to original version * sort suggestions * Revert "sort suggestions" This reverts commit 081c950. * Sort Co-authored-by: orenyodfat <[email protected]> * Bump version (#445) * Fix #447 (#449) * Fix #447 * Add test * Update Competition.spec.ts * Update Competition.spec.ts * Update Competition.spec.ts * Update Competition.spec.ts * Update Competition.spec.ts * Fix #446 * Fix * Fix * Update Competition.spec.ts * migration v38-2 * Update Competition.spec.ts * lint * Update mappings.json * Create model-ethdenver.json (#451) Co-authored-by: orenyodfat <[email protected]> * Competition ties (#453) * Fix competition ties handling * Fix * Fix * Revert "Fix" This reverts commit 5fe0be0. * Fix * Bump version * v37-1 (#454) * v37-1 * v39-2 * Update dummydao.json * migration v39-2 * Ethdenver bounty (#456) * Create ethdenver-bounty.json * Update ethdenver-bounty.json * add signal dao * remove ethdenver rionkeby Co-authored-by: benk10 <[email protected]> * Fix dxdao (#458) * workaround for dxdao * comment * Fix #455 (#457) * Fix #455 * Fix * Fix #459 (#460) * Fix #459 * Bump version * Update Competition.spec.ts * remove bitfwd,ethberlin,ethindia daos (#461) * remove bitfwd,ethberlin,ethidia daos * add jelle dao * spelling * BuffiDAO (#462) * BuffiDAO * sokol support * Update migration * Fix competition issue (#464) * Fix competition issue * bump version * v38-0 (#466) * v37-4 * Update migration * Update mappings.json * Update buffidao.json * Sponsor daos xdai * Fix #455 * Fix competition test * Update Competition.spec.ts * New sponsor daos (#471) * New sponsor daos * version tweak * More new daos * Tweak versions * New daos * Version tweak * Update migration and graph node versions (#472) * Update migration and graph node versions * Update compeition test version * Lint * Patch (#473) * Patch * Update mapping.ts * FIx * Fix test * Fix #468 (#475) Add "totalVotes" and "totalSubmissions" to CompetitionProposal * Fix #468 * Bump version * Adding 1 instead of 0 * Fix test string to number * Fix broken quey * New Ethdenver DAOs (#477) * New Ethdenver DAOs * More daos * Competition indexing (#478) * Fix #476 * Fix #470 * Bump version * Fix * Rename subbmisions -> suggestions * Rename * Rename * New fortmatic (#479) * New fortmatic * Update ethdenver-ethermint.json * Create _protocol.json (#480) Co-authored-by: benk10 <[email protected]> * Sportscasle dao (#481) * add sports castle dao * sportscastle dao * Delete _protocol.json * use try_parameters(... (#482) * use try_parameters(... * Add error field Co-authored-by: benk10 <[email protected]> * Update of _protocol.json (#484) * Create _protocol.json * generic scheme address updated (previous had wrong parameters) Co-authored-by: orenyodfat <[email protected]> Co-authored-by: benk10 <[email protected]> * Update dorg.json (#474) * Update dorg.json * update to new dao deployment Co-authored-by: benk10 <[email protected]> Co-authored-by: orenyodfat <[email protected]> * Create long-dao.json (#485) * init (#487) * Update long-dao.json (#486) * Add dxdao genericscheme ens .eth (#490) * Update of generic scheme on rinkeby by for undescore protocol (#489) * Create _protocol.json * generic scheme address updated (previous had wrong parameters) * new generic scheme with right params hash Co-authored-by: orenyodfat <[email protected]> Co-authored-by: benk10 <[email protected]> * Create cryptochicks.json (#488) * Create cryptochicks.json * Create cozdao.json * Create cryptochicks.json * Update utils * Updates * Update mappings * Update tests * Update tests * Fixed crash * More fixes * Fix competition test * Update Competition.spec.ts * Bump version Co-authored-by: pat-daostack <[email protected]> Co-authored-by: orenyodfat <[email protected]> Co-authored-by: Cesar Brazon <[email protected]> Co-authored-by: orishim <[email protected]> Co-authored-by: Jordan Ellis <[email protected]>
Right now we are seeing available redemptions for ContributionReward beneficiaries on proposals that failed by Expiring in the queue, this should not be the case.
The text was updated successfully, but these errors were encountered: