Update Magic Mirror Server and Client mode documentation

[briodan]

updating instructions on how to run MagicMirror in server only and client only mode.

fixes #3196

[sdetweil]

really there's just one install,
it copies the files and does NPM install

and then you can run any one of the three environments from there
full
or
server only
or
client only

just by changing the command

My script is targeted at running full but again all you have to do is change the command to launch the other environments

[briodan]

maybe i went a bit heavy on the documentation, my approach was with the mindset of what would have helped me figure it out without having to look at any other sources or do any googling.

Feel free to edit/amend as needed.

[bugsounet]

Can you change the target to the develop branch ?
It's better

Thanks :)

[briodan]

Can you change the target to the develop branch ? It's better

Thanks :)

done

[khassel]

@rejas how are develop and master working in this repo?

develop is behind master so reviewing stuff based on this is waste of time for me. Have we a concept how to handle diverging of these branches?

[rejas]

maybe we should merge master regularly back into develop (or everytime master gets some updates) so that this wont happen?

[khassel]

maybe we should merge master regularly back into develop (or everytime master gets some updates) so that this wont happen?

we should do it everytime master gets some updates, I did it now (with the expected result of merge conflicts in this PR).

[sdetweil]

we also need to remove the comment doesn't work over ssh.. as we fixed that a long time ago

[briodan]

resolved the conflicts and removed the ssh comment.

[khassel]

resolved the conflicts and removed the ssh comment.

Thanks!

As already mentioned in the comments above: The mm.sh file is removed on develop so installers/mm.sh will not exist anymore with next release. This file will be part of sam's script repo.

So I think the 2 sections beginning with To always start MagicMirror² in xxx mode you need to edit the installers/mm.sh file ... should be removed.

But I'm open to other approaches.

[briodan]

So should this really be a PR against develop? The intent of this PR when initially submitted was to document running server and client mode based on the way things worked at the time and I guess still will until the next release.

Since there is new functionality coming up in the future would it make sense to merge this into master to get current state documented, and setup another PR in develop to update the documentation based on new functionality?

Or would it make sense to just close this PR and set up a new one that accounts for functionality changes from scratch.

[khassel]

I would go for documentation for next release and omit the mm.sh stuff.

We can stay on develop and if it fits for master too I will "copy" it (in git wording: cherry pick) to master.

[rejas]

@briodan can you update your branch and resolve the conflicts so that we can get this PR into our code?

[sdetweil]

there is one install
and 3 execution modes
full , npm start , or npm run start
server, npm run server
client , npm run client

only the start script has a shortcut to not require the word run

installers/mm.sh was removed from the MagicMirror package and taken over by my scripted installer

[briodan]

This PR was written over a year ago to document how client and server mode were working at the time, with the Q4 release that changed the way things work for MM I don't know how much of this PR is still valid and I don't have the cycle to test it.

I updated my branch to latest, but beyond that I'll let others take this forward and update the documentation if still required.

[rejas]

I updated my branch to latest, but beyond that I'll let others take this forward and update the documentation if still required.

Thanks for your work, Ill take it upon me to finish the PR. Alas, you seem to have worked on your master branch and I cannot push to that it seems :-(