-
Notifications
You must be signed in to change notification settings - Fork 3.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add global environment variable feature #3941
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Generally, this looks great.
I've made some proposed changes - a couple bugs and message updates.
One piece I'd like to look at again is the edit dialog for the global-config
node. If a user manages to open it, I think it should help the user know what it contains and how to edit it.
I can create a more solid proposal this week.
packages/node_modules/@node-red/nodes/core/common/91-global-config.html
Outdated
Show resolved
Hide resolved
packages/node_modules/@node-red/editor-client/src/js/ui/env-var.js
Outdated
Show resolved
Hide resolved
packages/node_modules/@node-red/nodes/locales/en-US/common/91-global-config.html
Outdated
Show resolved
Hide resolved
packages/node_modules/@node-red/nodes/locales/en-US/messages.json
Outdated
Show resolved
Hide resolved
packages/node_modules/@node-red/nodes/locales/ja/common/91-global-config.html
Outdated
Show resolved
Hide resolved
…nfig.html Co-authored-by: Nick O'Leary <nick.oleary@gmail.com>
…r.js Co-authored-by: Nick O'Leary <nick.oleary@gmail.com>
…global-config.html Co-authored-by: Nick O'Leary <nick.oleary@gmail.com>
Co-authored-by: Nick O'Leary <nick.oleary@gmail.com>
…bal-config.html Co-authored-by: Nick O'Leary <nick.oleary@gmail.com>
Co-authored-by: Nick O'Leary <nick.oleary@gmail.com>
Co-authored-by: Nick O'Leary <nick.oleary@gmail.com>
packages/node_modules/@node-red/editor-client/src/js/ui/deploy.js
Outdated
Show resolved
Hide resolved
packages/node_modules/@node-red/nodes/core/common/91-global-config.html
Outdated
Show resolved
Hide resolved
packages/node_modules/@node-red/nodes/locales/en-US/messages.json
Outdated
Show resolved
Hide resolved
packages/node_modules/@node-red/editor-client/src/js/ui/env-var.js
Outdated
Show resolved
Hide resolved
packages/node_modules/@node-red/editor-client/locales/en-US/editor.json
Outdated
Show resolved
Hide resolved
packages/node_modules/@node-red/editor-client/locales/ja/editor.json
Outdated
Show resolved
Hide resolved
…r.json Co-authored-by: Nick O'Leary <nick.oleary@gmail.com>
…itor.json Co-authored-by: Nick O'Leary <nick.oleary@gmail.com>
Co-authored-by: Nick O'Leary <nick.oleary@gmail.com>
Co-authored-by: Nick O'Leary <nick.oleary@gmail.com>
…r.js Co-authored-by: Nick O'Leary <nick.oleary@gmail.com>
…nfig.html Co-authored-by: Nick O'Leary <nick.oleary@gmail.com>
Co-authored-by: Nick O'Leary <nick.oleary@gmail.com>
@HiroyasuNishiyama happy to get this merged now. Can you resolve the merge conflict on this branch? |
@knolleary Resolved the conflict. |
Proposed changes
This PR add global environment variable feature discussed at:
node-red/designs#76
This implementation introduces new special configuration node
global-config
and adds Environment tab to User settings menu.Checklist
grunt
to verify the unit tests pass