-
-
Notifications
You must be signed in to change notification settings - Fork 134
Add instructions for upgrading node when upgrade of MM fails #301
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
base: master
Are you sure you want to change the base?
Conversation
but you have to install n too. doesn't work on windows. |
I would guess there's a big chunk of users on debian or another similar linux flavor, and if they're all going to encounter this issue, I feel like we should include some kind of instruction. Even if it's noted that it only applies to linux install. |
my upgrade script does all that |
I think installing node is unrelated to mm, yes, it is required but the users should be able to google the things they need. Other thing is that there is not one method for installing node but several. I'm not for adding this to our docs here because it is outdated to fast. |
Which script is that? I'm just trying to make this as easy as possible on users. Yes, I was able to Google the answer pretty quickly, but I have to do that each time I upgrade MM because it's not something I commit to memory. If we put some instructions on the upgrade page, it makes this more user friendly. |
we can clearify docs and add e.g. an upgrade section, my point is that I don't want details concerning node in the docs which are changing to fast ... |
That's fine. There's already an upgrade section, we just need to point people to some resources on node so they don't have to go floating around to find answers themselves and potentially find bad info. For what it's worth, this was my upgrade experience:
|
so we could e.g. add a sentence here that maybe a nodejs update is required |
That's what this PR does. |
yes, but you are using a specific command for upgrading which maybe is not installed. I would like to see something like Your upgrade may fail if you do not have the version of |
my upgrade script is same place as install script |
and i post notes to the forum |
and for 2.31, which has instructions on how to get back if you accidentally did the git pull, and CANNOT upgrade node |
AND IF your MagicMirror implementation uses any modules that have gpio interaction, buttons, pir... you MAY have to run electron-rebuild on them and my upgrade script handles all that too |
I like the intention of the pull request. But I agree with @khassel, since there is no standard way for all platforms, we should not refer to a specific one in our documentation. I, for example, prefer Instead of recommending one way, it might make sense to recommend in our installation section to use a version manager for the node installation, which is offered on the official node website (https://nodejs.org/en/download). In the update section of our documentation we could then say that you should update using the version manager that was selected during node installation. |
I went looking on the Node repository for upgrade instructions, and they basically boil down to "just reinstall it". Let me review the resources Kristjan provided and I'll propose something new. It might not be for a bit, so if someone whats to jump on it, great. |
Perhaps the solution is to reference your upgrade script in the upgrade documentation, like your install script is referenced in the install instructions. I'll play with it and propose something based on the comments here. |
and just for completeness, if you didn't know, i try to get everyone to backup. not MagicMirror's job, but its good practices |
I tried again on a commit for this PR. I now realize that the upgrade script was referenced at the bottom of the page. This commit makes it a little more prominent, gives a little more info, and points directly to the anchor for the upgrade section of your scripts repository. Let me know if it now works. I back up my custom files offsite and on. You could blow a cannon through my Pi and I'd be back up and running within an hour of getting a new one. Edit: I just realized there's no mention of node upgrading under manual. Give me time to make a new commit before you review/approve/reject. |
Ok. Please let me know if I'm on the right track. |
I like the direction. I think it's great that the script exists and for some people it's certainly just the right thing, but since we want to encourage people to learn how the system works, I would somehow like to see that the documentation communicates that the manual way is the recommended way. What do you think about adding something like this in the script section: For users who prefer a straightforward and automated solution, this script offers a practical alternative. While convenient, using the script may mean you gain less insight into the system, potentially making it harder to troubleshoot issues independently. Or would that just be unnecessary bloat? I'm not so sure yet myself. |
I don't mind it, but you all are in charge. |
I am not in a position to make a decision here. I just try to contribute to the process to find a good solution 🙂 |
The last two times I have tried to upgrade MM, I have encountered errors because my node version was not correct. Thus, I think we should include information here about how one would properly go about upgrading node. I have included in this PR what works for me, but I am FAR from an expert in node.