Remove Wallet.ChainSynced/SetChainSynced APIs. #378

Merged
merged 2 commits into from Nov 9, 2016

Projects

None yet

3 participants

@jrick
Member
jrick commented Nov 9, 2016

Nothing ever set this value to true, and this was causing the wallet's
notification server to prematurely notify attached blocks, breaking
gRPC block notifications during chain switches and reorgs.

@jrick jrick Remove Wallet.ChainSynced/SetChainSynced APIs.
Nothing ever set this value to true, and this was causing the wallet's
notification server to prematurely notify attached blocks, breaking
gRPC block notifications during chain switches and reorgs.
8104017
@marcopeereboom

Oof. Good find.

@alexlyp
Member
alexlyp commented Nov 9, 2016

OK

@jrick jrick merged commit ec52a05 into decred:master Nov 9, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@jrick jrick deleted the jrick:rmsynced branch Nov 9, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment