-
Notifications
You must be signed in to change notification settings - Fork 5
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
Feat auto start #59
Feat auto start #59
Conversation
…n is started. issue with app.getName not working for Stele version
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.
I pulled down and tested on my Mac. The auto-start setting worked as intended, and the new setting is nicely positioned in the current form.
I'm requesting a single cleanup item before approving.
src/startupHelpers.js
Outdated
console.log("autolaunch enabled"); | ||
kioskAutoLaunch.enable() | ||
console.log(app.getName()); | ||
function whichSetting(settingsBool, configSetting) { |
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.
updated logging
Issue: #56 tested on Mac, Windows, and Ubuntu. Used node startup package. Had an issue using Linux AppImage, the workaround mentioned here placed the AppImage in the correct "~./config/autostart" but didn't start Stele. Added .zip build which works correctly with the node-auto-launch package.