Read the statement by Michael Teeuw here.
resolved.. MMM-NetworkScanner
-
@sonicgoose - I’m curious: why do you have commas inside the names of some of your devices? It looks like every Apple device is labeled “Apple,” with the comma inside. Same for the Google devices.
Also, could you expand on “doesn’t work properly”? What is it doing that you do not want? What is it not doing that you do want?
-
@bhepler The commas inside the quotes are a find and replace issue, easily fixed. The main problem is that this module says it offers different displays: list, columns, or columns with new lines—but all I get is the list. What I want is the columns with new lines, as shown on the module’s GitHub page (see link in first post.)
-
Double-check in the MMM-NetworkScanner.js file if you can find any comments with "sjj: " in them. If not then you have the original (ianperrin) version of the module, which doesn’t have the modifications in it such as the column mode.
Only the slametps fork has them, and that’s the version with the column view in it.
I hit this issue over the weekend and found that was the problem. Still not sure how I ended up with the wrong version, but I grabbed the correct script from the GitHub repo page and now it’s working fine (except it doesn’t work with my Apple devices as they don’t respond to pings).
-
git remote -vshows
origin https://github.com/spitzlbergerj/MMM-NetworkScanner.git (fetch) origin https://github.com/spitzlbergerj/MMM-NetworkScanner.git (push)but as you suggested I did find “sjj:” in some comments. Do you have a link to the fork that works for you? slametps doesn’t come up with anything on GitHub.
-
@sonicgoose its HIS fork and HIS special branch … so you have to clone HIS repo, then switch to the special branch that has the code
-
@sdetweil Thanks. Github is to me about as intuitive as theoretical physics. How exactly do I go about that?
-
@sonicgoose so, he said
Only the slametps fork has them, and that’s the version with the column view in it.that means the USER slametps has a copy (fork) of the original repo, and this fork has the changes
SO…
go to github and find the user slametps, (there is a search box) or google search ‘slametps github’, find his list of repos (push the repositories link), and scroll til you find it, or put ‘network’ (some part of the repo name) in the ‘find a repository’ field and hit enter
click on the repo name, click on clone or download, click on the clipboard (to copy the url to use in clone)
go to your system ~/MagicMirror/modules folder (in a terminal window)
rename your old MMM-NetworkScanner folder out of the way (to some other name, ANY other name)
then
git clone (paste the link from the clipboard)
cd MMM-NetworkScanner
it has a package.json, that means u need to do
npm installthen restart your MM, and read the readme for the slametps copy of the repo for new features
-
@sdetweil said in resolved.. MMM-NetworkScanner:
slametps
I’ve removed the version I had and installed the repo found here, which has outdated instructions telling me to install the ianperrin version. (I didn’t.) After restarting MM on my RPi, I know get just a list of mac addresses as I entered them in the config where I was getting at least a list of device names and icons. I’m getting really frustrated with this. Maybe time to take a break from it…
-
@sonicgoose i looked at his code, and it doesn’t look different…
I don’t know anything about the module
-
@sdetweil Thanks. I appreciate the help and that you took the time to look at the code. I wish I knew more about how this stuff works. I’m learning, but I’m also getting frustrated. I’ll revisit this another time.
