search

Achronite / mqtt-energenie-ener314rt

MQTT interface for Energenie ENER314-RT add-on board for the Raspberry Pi, designed for use by Home Assistant.
MIT License
13 stars 5 forks source link

Updated readme #45

Closed genestealer closed 10 months ago

genestealer commented 10 months ago

Added:

  1. Nodejs install details.
  2. Corrected typo for extracting zip: folder must be renamed.
genestealer commented 10 months ago

Oops, I just realised that the wording says that you need to have at least Node 16 installed. However my modified steps install Node 20x.

Considering you said only Node 16 is needed, I have changed NODE_MAJOR to 16, from 20. As the older versions of Linux do not support Node 18 and Node 20.

Achronite commented 10 months ago

Oops, I just realised that the wording says that you need to have at least Node 16 installed. However my modified steps install Node 20x.

Considering you said only Node 16 is needed, I have changed NODE_MAJOR to 16, from 20. As the older versions of Linux do not support Node 18 and Node 20.

I'm not sure I want this in the README, it adds a dependency between the pi distribution and the node version, so I'll have to keep modifying README to keep it up-to-date. I'm wondering if it might just be better to take all the node.js install instructions out altogether and add a link instead?

genestealer commented 10 months ago

Edited as per your feedback :)