Check the users balance before trying to publish/create a channel #794
|
@ -66,6 +66,9 @@
|
||||||
"perMachine": true
|
"perMachine": true
|
||||||
}
|
}
|
||||||
},
|
},
|
||||||
|
"scripts": {
|
||||||
|
"start": "./node_modules/.bin/electron src/main"
|
||||||
It does, I like to show the full path so others know exactly whats going on It does, I like to show the full path so others know exactly whats going on
I can change it to I can change it to `electron src/main` if you think that looks better
hmm no need, lets keep it as is then hmm no need, lets keep it as is then
|
|||||||
|
},
|
||||||
"devDependencies": {
|
"devDependencies": {
|
||||||
"devtron": "^1.4.0",
|
"devtron": "^1.4.0",
|
||||||
"electron": "^1.7.9",
|
"electron": "^1.7.9",
|
||||||
|
|
|
@ -1,5 +1,10 @@
|
||||||
Why format this? Also we should probably trust selector to fallback to 0? Why format this? Also we should probably trust selector to fallback to 0?
Ignore this for now. Just copy/pasted from somewhere else Ignore this for now. Just copy/pasted from somewhere else
Why format this? Also we should probably trust selector to fallback to 0? Why format this? Also we should probably trust selector to fallback to 0?
Ignore this for now. Just copy/pasted from somewhere else Ignore this for now. Just copy/pasted from somewhere else
|
|||||||
import React from "react";
|
import React from "react";
|
||||||
import { connect } from "react-redux";
|
import { connect } from "react-redux";
|
||||||
import PublishForm from "./view";
|
import PublishForm from "./view";
|
||||||
|
import { selectBalance } from "redux/selectors/wallet";
|
||||||
Why format this? Also we should probably trust selector to fallback to 0? Why format this? Also we should probably trust selector to fallback to 0?
Ignore this for now. Just copy/pasted from somewhere else Ignore this for now. Just copy/pasted from somewhere else
|
|||||||
|
|
||||||
export default connect(null, null)(PublishForm);
|
const select = state => ({
|
||||||
Why format this? Also we should probably trust selector to fallback to 0? Why format this? Also we should probably trust selector to fallback to 0?
Ignore this for now. Just copy/pasted from somewhere else Ignore this for now. Just copy/pasted from somewhere else
Why format this? Also we should probably trust selector to fallback to 0? Why format this? Also we should probably trust selector to fallback to 0?
Ignore this for now. Just copy/pasted from somewhere else Ignore this for now. Just copy/pasted from somewhere else
|
|||||||
|
balance: selectBalance(state),
|
||||||
Why format this? Also we should probably trust selector to fallback to 0? Why format this? Also we should probably trust selector to fallback to 0?
Ignore this for now. Just copy/pasted from somewhere else Ignore this for now. Just copy/pasted from somewhere else
|
|||||||
|
});
|
||||||
Why format this? Also we should probably trust selector to fallback to 0? Why format this? Also we should probably trust selector to fallback to 0?
Ignore this for now. Just copy/pasted from somewhere else Ignore this for now. Just copy/pasted from somewhere else
|
|||||||
|
|
||||||
Why format this? Also we should probably trust selector to fallback to 0? Why format this? Also we should probably trust selector to fallback to 0?
Ignore this for now. Just copy/pasted from somewhere else Ignore this for now. Just copy/pasted from somewhere else
|
|||||||
|
export default connect(select, null)(PublishForm);
|
||||||
Why format this? Also we should probably trust selector to fallback to 0? Why format this? Also we should probably trust selector to fallback to 0?
Ignore this for now. Just copy/pasted from somewhere else Ignore this for now. Just copy/pasted from somewhere else
|
|||||||
|
|
||||||
Why format this? Also we should probably trust selector to fallback to 0? Why format this? Also we should probably trust selector to fallback to 0?
Ignore this for now. Just copy/pasted from somewhere else Ignore this for now. Just copy/pasted from somewhere else
Why format this? Also we should probably trust selector to fallback to 0? Why format this? Also we should probably trust selector to fallback to 0?
Ignore this for now. Just copy/pasted from somewhere else Ignore this for now. Just copy/pasted from somewhere else
|
|
@ -48,11 +48,22 @@ class ChannelSection extends React.PureComponent {
|
||||||
|
|
||||||
handleNewChannelBidChange(event) {
|
handleNewChannelBidChange(event) {
|
||||||
this.setState({
|
this.setState({
|
||||||
newChannelBid: event.target.value,
|
newChannelBid: parseFloat(event.target.value),
|
||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
|
||||||
handleCreateChannelClick(event) {
|
handleCreateChannelClick(event) {
|
||||||
|
const { balance } = this.props;
|
||||||
|
const { newChannelBid } = this.state;
|
||||||
|
|
||||||
|
if (newChannelBid > balance) {
|
||||||
|
this.refs.newChannelName.showError(
|
||||||
|
__("Unable to create channel due to insufficient funds.")
|
||||||
|
);
|
||||||
|
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
this.setState({
|
this.setState({
|
||||||
creatingChannel: true,
|
creatingChannel: true,
|
||||||
});
|
});
|
||||||
|
|
|
@ -61,6 +61,15 @@ class PublishForm extends React.PureComponent {
|
||||||
}
|
}
|
||||||
|
|
||||||
handleSubmit() {
|
handleSubmit() {
|
||||||
|
const { balance } = this.props;
|
||||||
|
const { bid } = this.state;
|
||||||
|
|
||||||
|
if (bid > balance) {
|
||||||
|
this.handlePublishError({ message: "insufficient funds" });
|
||||||
|
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
this.setState({
|
this.setState({
|
||||||
submitting: true,
|
submitting: true,
|
||||||
});
|
});
|
||||||
|
|
does it work to just say "electron src/main" since electron is in the bin?