Skip to content

Update To enable unit numbers to pass to emoncms as node numbers#4

Closed
rmtucker wants to merge 3 commits into
letscontrolit:masterfrom
rmtucker:master
Closed

Update To enable unit numbers to pass to emoncms as node numbers#4
rmtucker wants to merge 3 commits into
letscontrolit:masterfrom
rmtucker:master

Conversation

@rmtucker
Copy link
Copy Markdown
Contributor

Sorry about this Afterthought.
This enables the ESPEasy unit number to be passed to emoncms as a node number.
This means in Emoncms that every Unit will be on a seperate Node.

rmtucker added 2 commits October 11, 2015 13:32
This small change passes the unit number to emoncms node number
Which enables every unit to be a seperate node in emoncms.
Sorry this was an afterthought.
@rmtucker
Copy link
Copy Markdown
Contributor Author

I have added a few setup instructions Here:-
http://www.esp8266.nu/forum/viewtopic.php?f=6&t=169
Thought it may help.

@ghost
Copy link
Copy Markdown

ghost commented Oct 11, 2015

It seems you're one commit behind. Don't know what will happen if I merge this pull request right now.
(we have been on svn instead of github for years..., so all this pushing/pulling/merging is new to me...)

@rmtucker
Copy link
Copy Markdown
Contributor Author

It has got me completely lost (First time User of Github.
Can you still merge the changes?
If not i will clear my repository and fork it again and redo the changes and re-commit.?

@rmtucker rmtucker closed this Oct 11, 2015
Grovkillen added a commit that referenced this pull request Apr 24, 2018
TD-er pushed a commit that referenced this pull request May 13, 2018
Sat May 12 04:00:06 CEST 2018
@svmac svmac mentioned this pull request Nov 7, 2018
14 tasks
TD-er pushed a commit that referenced this pull request Jan 7, 2019
TD-er pushed a commit that referenced this pull request Feb 1, 2019
TD-er pushed a commit that referenced this pull request Feb 24, 2020
TD-er pushed a commit that referenced this pull request Mar 27, 2020
@stefbo27 stefbo27 mentioned this pull request May 18, 2020
TD-er pushed a commit that referenced this pull request Oct 22, 2020
TD-er pushed a commit that referenced this pull request May 10, 2021
[Custom build] Allow for more styling options in custom build
Brommander added a commit to Brommander/ESPEasy that referenced this pull request May 13, 2026
letscontrolit#1  Translate all comments to English
letscontrolit#2  Use const reference and member initializer list in constructors
letscontrolit#3  Explicit {} initialization for global arrays (clarity)
letscontrolit#4  Explicit {} initialization for global int variables (clarity)
letscontrolit#5  Use auto& dev and DEVICE_TYPE_SERIAL in PLUGIN_DEVICE_ADD
letscontrolit#6  Use const variables for serial port/pins from CONFIG
letscontrolit#7  Add bounds checks before array index access
letscontrolit#8  Replace millis() arithmetic with timePassedSince()
letscontrolit#9  Add bounds check at top of D0 read loop
letscontrolit#11 Replace C-style casts with static_cast<>
letscontrolit#12 Pre-calculate scale factor (powf) in state 32, not per value
letscontrolit#13 Declare loop variable outside scope for use in debug log
letscontrolit#14 Remove hardcoded serial pins — port now configured via UI
Brommander added a commit to Brommander/ESPEasy that referenced this pull request May 13, 2026
letscontrolit#1 Better description of models
letscontrolit#2 reduce models to 2
letscontrolit#3 Logging nullptr with timer
letscontrolit#4 use memcmp instead of long if
letscontrolit#5 better check of lenght and memory access with functions
Brommander added a commit to Brommander/ESPEasy that referenced this pull request May 13, 2026
letscontrolit#1 Better description of models
letscontrolit#2 reduce models to 2
letscontrolit#3 Logging nullptr with timer
letscontrolit#4 use memcmp instead of long if
letscontrolit#5 better check of lenght and memory access with functions
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant