-
Notifications
You must be signed in to change notification settings - Fork 89
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
TX confirmation modal 2.0 #742
Conversation
*updated AddressHelper utilites *added send information from send component to confirmation component *added variables to html
Waiting for #779 to be merged |
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.
Looks good to me (after the fiat hidding).
& > .currency { | ||
font-size: 17px; | ||
} | ||
& > .fiat { |
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.
Can we hide the fiat div via css rather than comments?
Easier later on & cleaner.
When using the Amount class, make sure to initialize the variable with |
fix ringsize bug test things
ringsize: number, numsignatures: number, substractfeefromamount: boolean) { | ||
// @TODO: estimate fee should be change | ||
public sendTransaction(tx: TransactionBuilder) { | ||
tx.comment = tx.narration = tx.note; |
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.
Need to check this out..
759c029
to
099687f
Compare
Hye @kewde @anandsinghparihar when I started the electron for this branch and got this issue (OSX) |
@vikas-cis could you please debug that OSX error window ? |
@anandsinghparihar yeah I am checking it :) Thanks |
#903 log issue has been fix here Thanks |
@@ -199,6 +199,7 @@ export class SendComponent implements OnInit { | |||
} | |||
|
|||
dialogRef.componentInstance.dialogContent = txt; | |||
console.log('>>>>???componentInstance????????>', this.send); |
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.
can you remove the console log @anandsinghparihar
9749962
to
a4b9a7f
Compare
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.
looks good to me (y)
Note: this PR was based on @pciavald's #724 because I couldn't access the Send tab due to the bug on
dev
. If needed, I can rebase ondev
when fixed.TODO