Skip to content

Conversation

drmibell
Copy link
Contributor

@drmibell drmibell commented Aug 4, 2022

This PR closes issue #284. It adds documentation for features added since version 2.0 and should be checked against PR #264 and #273.

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Aug 4, 2022

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: drmibell / name: Michael I Bell (ae80bd3)


Since [version 2.1](https://nodered.org/blog/2021/10/21/version-2-1-released#flowgroup-level-environment-variables), environment variables can be set at the flow or group level. This is done in the appropriate tab in the edit dialog for the flow or group.

### Raspberry Pi OS considerations
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

While the script is mainly aimed at Pi users, it works well for any Debian based install eg Ubuntu. So this section may need renaming.

Copy link
Member

@knolleary knolleary left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some proposed changes to the wording - keeping the Since Node-RED x.y.z formatting consistent with other parts of the docs.

@knolleary knolleary merged commit 4aee6fe into node-red:master Aug 4, 2022
@knolleary
Copy link
Member

Thanks - did a bit of rework to account for the other PRs that updated this page. Appreciate you taking the time to do this.

@drmibell
Copy link
Contributor Author

drmibell commented Aug 7, 2022

I just noticed that the code block in the Template node section does not render properly on the website. The source should probably be

{% raw %}My favourite colour is {{env.COLOUR}}.{% endraw %}

(Needed, I assume, to escape the double curly braces and one of those subtle differences between html and GitHub markdown known only to real professionals...)

@dceejay
Copy link
Member

dceejay commented Aug 19, 2022

Fixed by 670a6f3

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.

3 participants