add ui component for minting txn #6
Conversation
Signed-off-by: eolesinski <[email protected]>
| <b-form-label for="amount-input">Number of UTXOs to Mint</b-form-label> | ||
| </div> | ||
|
|
||
| <!-- form input area for specifying the amount of utxos to mint --> |
There was a problem hiding this comment.
nit: Ideally, the arrows provided here cannot even generate a negative number
There was a problem hiding this comment.
The regexp in numberFormatValidator below seems to require the string start with a digit. (Non-negative) should also check that this isn't a bug server side though as well.
There was a problem hiding this comment.
amountFormatValidator requires > 0 as well.
There was a problem hiding this comment.
Not an issue for this PR but I'm a little worried that we are allowing unauthenticated posts to the mintTx endpoint - should probably rearchitect things at some point.
There was a problem hiding this comment.
opened #10 to capture this.
|
also, open to feedback from @anders94 as well! |
|
LGTM |
|
As noted in a code level comment, at some point we should propose at least a minimal authentication system so these endpoints aren't just reachable by anyone. Should also do the same with a minimal key service that does something better than hardcode minting keys into the server. The risk we run into here is a less than observant user might assume this level of security is suggested by us which it very much is not! |
This PR adds a UI component (an Admin page) to facilitate initiating new minting transactions. This is the first part of connecting the wallet UI to opencbdc-tx as detailed in #2.
Default page:
Input validation checks: