-
-
Notifications
You must be signed in to change notification settings - Fork 629
refactor(ui): Cleanup template option handling #1430
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
base: master
Are you sure you want to change the base?
refactor(ui): Cleanup template option handling #1430
Conversation
7e2cc8b to
19352e3
Compare
7068b86 to
4590544
Compare
| "env": { | ||
| "node": true | ||
| "node": true, | ||
| "es2024": true |
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.
I'm not sure about this yet. Without this eslint complains about globalThis not being defined. But I still haven't understood the impact of the change and which version to choose.
|
/regenerate-static-assets |
|
👋 PythonGermany, seems as if you have not write/admin permissions in this repo, permissions: read |
|
@TwiN Just as I had worried about, the rights for me are obviously not the same as in my fork which prevents me from running the command, even as author of the PR 😞 Can you test if you can trigger it in this PR to commit to the branch? We might have to whitelist the author of this PR using their username after all. |
Summary
In the development environment the UI config placeholders are not replaced.
Improves how configs which have template placeholders as default value (replaced at go application startup by actual config value, as far as I know) are initialized and accessed in the web app.
Checklist
README.md, if applicable.