Skip to content

Correct Default Proxy header description on ProxySettings - Follow-up #391 #395

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

Merged

Conversation

pablomartin4btc
Copy link
Contributor

@pablomartin4btc pablomartin4btc commented Apr 8, 2024

This is a follow-up from #391 (comment).

Currently/ main branch screenshot.

Screenshot from 2024-04-07 23-00-31

This PR branch screenshot.

image

@pablomartin4btc
Copy link
Contributor Author

Gentle reminder to #391's reviewers: @johnny9, @GBKS, @D33r-Gee, @MarnixCroes, @stackingsaunter.

Copy link
Contributor

@MarnixCroes MarnixCroes left a comment

Choose a reason for hiding this comment

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

lgtm

maybe Tor Proxy description could be changed also, to
Run Tor connections through a dedicated proxy. to be consistent in the wording and the enable to is a bit redundant as well

Follow-up from bitcoin-core#391 (comment).

Removing redundant Tor Proxy description from the top Default Proxy header label.

Rewording Tor Proxy description to be consistent with the top header label.
@pablomartin4btc pablomartin4btc force-pushed the qml-ipaddressvalueinput-followup branch from 3495832 to d558b05 Compare April 8, 2024 15:06
@pablomartin4btc
Copy link
Contributor Author

Updates:

Copy link
Contributor

@D33r-Gee D33r-Gee left a comment

Choose a reason for hiding this comment

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

tACK d558b05

lgtm, wording is clear...

Ubuntu 22.04 Screenshot

Screenshot 2024-04-08 084731

Android (armv7) Screenshot

Screenshot_20240408_085820

Copy link
Contributor

@MarnixCroes MarnixCroes left a comment

Choose a reason for hiding this comment

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

ack d558b05

@GBKS
Copy link
Contributor

GBKS commented Apr 9, 2024

tACK d558b05

@hebasto hebasto merged commit b80e167 into bitcoin-core:main Apr 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants