Skip to content
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

NR theme and node failure #436

Closed
LazzaAU opened this issue Apr 13, 2021 · 3 comments
Closed

NR theme and node failure #436

LazzaAU opened this issue Apr 13, 2021 · 3 comments
Assignees
Labels
bug Something isn't working

Comments

@LazzaAU
Copy link
Contributor

LazzaAU commented Apr 13, 2021

During a new install, i've noticed NR sometimes fails to install the dark theme, she also on occasions has failed to configure the alice nodes and i have to go to the pallette to enable them. For me the theme part has happened at least 4 times. Other reported cases of this in discord also but they choose not to add a github issue :).

Suspected cause

I feel the issue lays with user/alice rebooting during final stages of the NR configuration.

My thoughts

NR can take a little while to install and its possible Alice gets rebooted before NR has been configured fully due to user forgetting NR is still doing it's thing in the background ?.
Senario i suspect is the "package.json" file gets installled and is part way through the remaining code when alice is potentially restarted ? So now on startup Alice see's package.json file and therefore believes NR is all good to go so ignores the rest of the code despite settings.js file not being properly configured.

**Potential Remedy **

Are you happy for me to add a boolean in config.json called nodeRedConfigured = false . then check that key on restart... if False, re run configureNewNodeRed(self) if True ignore ?

@Psychokiller1888
Copy link
Member

Hmmm, I'd rather make the node red install run on the main thread, so it blocks any further instructions and Alice doesn't reboot while it is installing it

Psychokiller1888 added a commit that referenced this issue Apr 14, 2021
@Psychokiller1888 Psychokiller1888 self-assigned this Apr 14, 2021
@Psychokiller1888 Psychokiller1888 added the bug Something isn't working label Apr 14, 2021
@LazzaAU
Copy link
Contributor Author

LazzaAU commented Apr 14, 2021

i did appear to fix the theme side of it by adding a json file in system/nodeRed.

if key in json file = false, apon second restart and package.json file exists, then reconfigure NR during bootup, else do nothing

However looks like your all over it so will leave it alone :)

@philipp2310
Copy link
Member

continued in #617

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants