8000 Checkbox to open PowerShell after the (Windows) installer has finished by bergmeister · Pull Request #4203 · PowerShell/PowerShell · GitHub
[go: up one dir, main page]

Skip to content

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

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
merged 2 commits into from
Jul 15, 2017

Conversation

bergmeister
Copy link
Contributor
@bergmeister bergmeister commented Jul 9, 2017

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
Copy link
msftclas commented Jul 9, 2017

@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
@@ -31,6 +31,13 @@

<!-- Set properties for add/remove programs -->
<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
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
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.

Copy link
Member
@SteveL-MSFT SteveL-MSFT left a comment

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
0