Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Checkbox to open PowerShell after the (Windows) installer has finished#4203

Merged
daxian-dbw merged 2 commits into
PowerShell:masterPowerShell/PowerShell:masterfrom
bergmeister:masterbergmeister/PowerShell:masterCopy head branch name to clipboard
Jul 15, 2017
Merged

Checkbox to open PowerShell after the (Windows) installer has finished#4203
daxian-dbw merged 2 commits into
PowerShell:masterPowerShell/PowerShell:masterfrom
bergmeister:masterbergmeister/PowerShell:masterCopy head branch name to clipboard

Conversation

@bergmeister

@bergmeister bergmeister commented Jul 9, 2017

Copy link
Copy Markdown
Contributor

Added a checkbox (unchecked by default) to the last dialogue of the Windows installer to provide the option of opening PowerShell because that's what most people want to do if they installed PowerShell.

@msftclas

msftclas commented Jul 9, 2017

Copy link
Copy Markdown

@bergmeister,
Thanks for having already signed the Contribution License Agreement. Your agreement has not been validated yet. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@bergmeister bergmeister changed the title Checkbox to open PowerShell after the (Windows) installation Checkbox to open PowerShell after the (Windows) installer has finished Jul 9, 2017
@SteveL-MSFT SteveL-MSFT requested a review from raghushantha July 9, 2017 15:56
Comment thread assets/Product.wxs Outdated
<Property Id="ARPHELPLINK" Value="$(var.InfoURL)" />

<!-- Checkbox to allow starting PowerShell after the installation (in UI mode only) -->
<Property Id="WIXUI_EXITDIALOGOPTIONALCHECKBOXTEXT" Value="Open PowerShell" />

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Replace PowerShell with $(env.ProductName)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Good idea. Done and manually tested on the MSI artifact of CI to verify the CheckBox text is still 'Open PowerShell' after using a variable for it.

@SteveL-MSFT SteveL-MSFT left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for the contribution! LGTM

@daxian-dbw daxian-dbw merged commit f95fbf1 into PowerShell:master Jul 15, 2017
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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.