Skip to content

[GUI] Fix veil.tools hyperlink in wallet#1034

Merged
seanPhill merged 1 commit intoVeil-Project:masterfrom
Necrosys:fix_issue_1032
Jun 1, 2023
Merged

[GUI] Fix veil.tools hyperlink in wallet#1034
seanPhill merged 1 commit intoVeil-Project:masterfrom
Necrosys:fix_issue_1032

Conversation

@Necrosys
Copy link
Contributor

@Necrosys Necrosys commented Jun 1, 2023

To open a URL in a browser, setOpenExternalLinks for label_3 must set to true.

Copy link
Collaborator

@ohcee ohcee left a comment

Choose a reason for hiding this comment

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

utACK Although needs to be tested after the workflow gets approved

@seanPhill
Copy link
Collaborator

Approved the workflow. Let's see! :)

@ohcee
Copy link
Collaborator

ohcee commented Jun 1, 2023

ACK works well for me on MacOS.

Copy link
Collaborator

@seanPhill seanPhill left a comment

Choose a reason for hiding this comment

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

Confirmed working in macOS and Ubuntu.

@seanPhill seanPhill merged commit ab9c6f5 into Veil-Project:master Jun 1, 2023
@seanPhill
Copy link
Collaborator

Thanks! :)

@Necrosys Necrosys deleted the fix_issue_1032 branch June 2, 2023 02:12
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.

3 participants