Skip to content
This repository has been archived by the owner on Jun 4, 2018. It is now read-only.

Refresh ticket price and retarget without refresh button. #244

Merged
merged 2 commits into from
Apr 26, 2017

Conversation

jrick
Copy link
Member

@jrick jrick commented Apr 24, 2017

Fixes #226.

NOTE: This is rebased over #241.

This updates the stakepool client to support the v2 API which can be
used to set vote preferences, and updates to a newer version of the
wallet gRPC API to support saving and retreiving vote preferences from
the wallet DB.

All agendas in the latest support stake version are now shown in the
purchase tickets view with dropdown boxes for users to select their
vote preference on an agenda.  Whenever a choice is changed, the local
vote preferences are saved by the wallet and each configured
stakepool is updated through the API.

Note that vote preferences are never read through the stakepool API.
If a user changes their preferences through another channel (e.g. the
stakepool web UI, or other wallet software) the update will not be
visible in Paymetheus and Paymetheus may overwrite preferences when
additional modifications are made.  This is unfortunate but I don't
see a better way of handling this considering that multiple stakepools
may be configured, each with different vote preferences.

Closes #240.
Copy link
Member

@marcopeereboom marcopeereboom left a comment

Choose a reason for hiding this comment

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

Very nice. Retarget interval changes at roughly the same time as the wallet.

@jrick jrick merged commit 2aed10e into decred:master Apr 26, 2017
@jrick jrick deleted the refresh branch April 26, 2017 14:54
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.

2 participants