Skip to content

Conversation

@hebasto
Copy link
Member

@hebasto hebasto commented Sep 7, 2022

An unneeded character slipped in #629.

Copy link
Contributor

@jarolrod jarolrod left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK 5f28fc8

translation source file diff shows's a + sign where the extra space is, this PR removes it:

diff --git a/src/qt/locale/bitcoin_en.ts b/src/qt/locale/bitcoin_en.ts
index 883d3fe24..5a974826d 100644
--- a/src/qt/locale/bitcoin_en.ts
+++ b/src/qt/locale/bitcoin_en.ts
@@ -751,7 +751,7 @@ Signing is only possible with addresses of the type &apos;legacy&apos;.</source>
     <message>
         <location line="+8"/>
         <source>Restore Wallet</source>
-        <extracomment>Title of pop-up window shown when the user is attempting to + restore a wallet.</extracomment>
+        <extracomment>Title of pop-up window shown when the user is attempting to restore a wallet.</extracomment>
         <translation type="unfinished"></translation>
     </message>
     <message>

Copy link
Member

@jonatack jonatack left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK 5f28fc8

@hebasto hebasto merged commit bc29c4d into bitcoin-core:master Sep 14, 2022
@hebasto hebasto deleted the 220907-tr branch September 14, 2022 13:57
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Sep 14, 2022
5f28fc8 qt: Cleanup translation comment (Hennadii Stepanov)

Pull request description:

  An unneeded character slipped in bitcoin-core/gui#629.

ACKs for top commit:
  jarolrod:
    ACK 5f28fc8
  jonatack:
    utACK 5f28fc8

Tree-SHA512: 210fb626e8035786cf6859160c60b2815c813e02908c75efc71a2c64d511edd6f81b2f67f1c98b29122b990260ebf663da445ea2d01b6268e3e046ada1ca5b6e
@bitcoin-core bitcoin-core locked and limited conversation to collaborators Sep 14, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants