Skip to content

explain refresh parameter of mqtt activity message with examples#17

Merged
lanrat merged 2 commits intolanrat:mainfrom
elratt0r:explain_refresh_parameter
Dec 10, 2022
Merged

explain refresh parameter of mqtt activity message with examples#17
lanrat merged 2 commits intolanrat:mainfrom
elratt0r:explain_refresh_parameter

Conversation

@elratt0r
Copy link
Copy Markdown
Contributor

Examples for PR #13

@elratt0r elratt0r force-pushed the explain_refresh_parameter branch from f1cff38 to dfe2f5d Compare December 10, 2022 22:49
@lanrat
Copy link
Copy Markdown
Owner

lanrat commented Dec 10, 2022

Looks good, but you forgot to update the MQTT topic from the changes you made for MQTT_NODE_ID in #16 ;)

@lanrat lanrat closed this Dec 10, 2022
@lanrat lanrat reopened this Dec 10, 2022
@lanrat
Copy link
Copy Markdown
Owner

lanrat commented Dec 10, 2022

Clicked the wrong button.

@elratt0r
Copy link
Copy Markdown
Contributor Author

Sorry, I based it on the actual main state, that had not included the use_node_id yet. Should have waited a minute for you to pull :-)

@elratt0r elratt0r force-pushed the explain_refresh_parameter branch from dfe2f5d to fcababa Compare December 10, 2022 22:54
@lanrat lanrat merged commit 0fc1fe4 into lanrat:main Dec 10, 2022
@elratt0r elratt0r deleted the explain_refresh_parameter branch December 10, 2022 23:04
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.

2 participants