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

Add expired tickets infrastructure #44

Closed
wants to merge 1 commit into from
Closed

Add expired tickets infrastructure #44

wants to merge 1 commit into from

Conversation

alexlyp
Copy link
Member

@alexlyp alexlyp commented Oct 20, 2016

This still requires wallet PR to discern missed PoolTickets vs expired
Fixes #57

@chappjc
Copy link
Member

chappjc commented Oct 24, 2016

tACK (what a challenge with 3 other PRs!)
Also added an expired field on the /stats page:
pull/44/head...chappjc:expired-ref

@alexlyp
Copy link
Member Author

alexlyp commented Oct 24, 2016

Though this should be ready to go, I still need to add expired to the pooltickets (which requires a db update in dcrwallet) Going to wait to merge until I can fully test with that

@jolan
Copy link
Contributor

jolan commented Feb 17, 2017

Well this needs to be rebased obviously. Aside from that, decred/dcrwallet#371 needs to be decided/fixed/implemented before this can go in.

@alexlyp
Copy link
Member Author

alexlyp commented Feb 17, 2017

Hmmm apparently my branch is lost to the ether, I'll close this and reopen one once the changes hit dcrwallet

@alexlyp alexlyp closed this Feb 17, 2017
@chappjc
Copy link
Member

chappjc commented Feb 17, 2017

Weird, I cherry picked your commit long ago, just waiting for wallet to support it.

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.

3 participants