-
Notifications
You must be signed in to change notification settings - Fork 11
Update MrCoin exchange links #673
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
Conversation
|
Deploy preview for augmint ready! Built with commit a4ec0d8 |
phraktle
left a comment
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.
@ejulcsi please restore the address and amount parameters in the link.
phraktle
left a comment
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.
Pass address / amount based on this document: https://gist.github.com/debreczeni/0e9b7bfc6d9e32715db63aa937f7cb18
src/components/BaseComponents.js
Outdated
|
|
||
| minMrCoinAEurAmount: value => { | ||
| return value < 15 ? `Amount must be at least: 15 A-EUR` : undefined; | ||
| minMrCoinAmount: (minvalue, currency) => value => { |
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.
should be just minAmount
| this.state = { | ||
| orderDirection: ADDFUND, | ||
| amount: "", | ||
| minEUR: Validations.minMrCoinAmount(FUNDS[0].eurLimit, "EUR"), |
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.
should not be in state (use a const or move it to funds.js)
No description provided.