-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
backport: Merge bitcoin/bitcoin#22337 #6130
Conversation
6b7c87b
to
9c9676b
Compare
This pull request has conflicts, please rebase. |
8413f62
to
36c5569
Compare
Hello @UdjinM6, @knst , @PastaPastaPasta requesting review |
Hello @UdjinM6 , @PastaPastaPasta requesting review |
Needs rebase to fix |
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.
utACK 0f7d842
For some reason you rebased this on a commit from like a month ago? can you rebase on latest develop? I like the added signature though! Note: the rebase to 62b11c4 looks clean, commit is same, but I'd prefer not merging in from a really old based branch |
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.
utACK c8ae089
rebase is clean
rebased from latest , @UdjinM6 had approved the changes earlier , rebasing removed the approval, I will request again |
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.
utACK c8ae089
92993aa Change SignTransaction's input_errors to use bilingual_str (Andrew Chow) 171366e Use bilingual_str for address fetching functions (Andrew Chow) 9571c69 Add bilingual_str::clear() (Andrew Chow) Pull request description: In a couple of places in the wallet, errors are `std::string`. In order for these errors to be translated, change them to use `bilingual_str`. ACKs for top commit: hebasto: re-ACK 92993aa, only rebased since my [previous](bitcoin#22337 (review)) review, verified with klementtan: Code review ACK 92993aa meshcollider: Code review ACK 92993aa Tree-SHA512: 5400e419dd87db8c49b67ed0964de2d44b58010a566ca246f2f0760ed9ef6a9b6f6df7a6adcb211b315b74c727bfe8c7d07eb5690b5922fa5828ceef4c83461f Signed-off-by: Vijay <vijaydas.mp@gmail.com>
bitcoin backport