-
Notifications
You must be signed in to change notification settings - Fork 120
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 basic autobuyer start/stop #325
Conversation
app/reducers/control.js
Outdated
startAutoBuyerError: null, | ||
startAutoBuyerSuccess: action.success, | ||
startAutoBuyerRequestAttempt: false, | ||
startAutoBuyerResponse: action.purchaseTicketsResponse, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wrong action field
app/reducers/control.js
Outdated
stopAutoBuyerError: null, | ||
stopAutoBuyerSuccess: action.success, | ||
stopAutoBuyerRequestAttempt: false, | ||
stopAutoBuyerResponse: action.purchaseTicketsResponse, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
another
Fixed the two errors pointed out but going to hook up to minimal part of gui before enxt review. |
f86257d
to
335e49d
Compare
app/components/SideBar.js
Outdated
@@ -10,6 +10,7 @@ import './fonts.css'; | |||
function mapStateToProps(state) { | |||
return { | |||
walletService: state.grpc.walletService, | |||
ticketBuyerService: state.grpc.ticketBuyerService, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmmm don't think you need the ticketBuyerService here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll remove. That was when I was getting a little desperate trying to find where I was missing something.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed.
Needs more styling of course, but should be good to on in after review now. |
OK |
Takes care of the first part of #268