-
Notifications
You must be signed in to change notification settings - Fork 112
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
Enable the Workstation App to run at boot on Windows #671
Conversation
|
||
<ComponentRef Id="ChefWSAppAutostartShortcut" /> | ||
--> | ||
<ComponentRef Id="ChefWSAppAutostartShortcut" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this will start the Chef Workstation App? Maybe we need a build to verify this functionality?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If everyone is ok with the concept I'll kick off an adhoc
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok, let us wait.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Waiting on validation for this one - want to try installing it on a Windows desktop under a non-admin user |
6b593ab
to
6c53f2c
Compare
6c53f2c
to
744c47b
Compare
We will review this PR and asses whether we can merge this in the current sprint without having feature parity with MacOS / *nix. Is it even possible, in a future PR, to get feature parity for this in *nix? |
If we want to keep folks up to date we should run this at boot. It seems acceptable to have it do so on Windows while not doing it on every other platform. Signed-off-by: Tim Smith <tsmith@chef.io>
744c47b
to
331bf30
Compare
…o windows_at_boot
rebased from master and launched an ad hoc build |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let me see if I can find an easy trick to start the WS app after install.
Signed-off-by: Salim Afiune <afiune@chef.io>
Signed-off-by: Salim Afiune <afiune@chef.io>
33ab52d
to
fdaeaca
Compare
One more ad hoc build 🤞 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ship it!
@tas50 indeed! Just waiting for the ad hoc build and a quick test! |
Signed-off-by: Salim Afiune <afiune@chef.io>
052ce2e
to
1404a69
Compare
This was probably a 3 or 5 from a point perspective. |
If we want to keep folks up to date we should run this at boot. It seems
acceptable to have it do so on Windows while not doing it on every other
platform.
GH Issue: #819
Signed-off-by: Tim Smith tsmith@chef.io