Skip to content
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

UI config now primarily pulled from backend endpoint #4273

Merged
merged 6 commits into from
Jan 30, 2024

Conversation

EricWittmann
Copy link
Member

@EricWittmann EricWittmann commented Jan 29, 2024

I have introduced a new REST API endpoint to return the UI config:

/system/uiConfig

The response payload is a JSON document containing all of the UI config (except the Registry v3 endpoint for obvious reasons).

The UI now only needs to be configured with the v3 endpoint. It will then fetch the baseline config from the above system endpoint. All config can now be set on the backend pod/container via env variables. A few new env vars have been introduced to help with this.

All config can be overridden using env vars in the UI pod/container. The UI will merge the config from the endpoint with the local overrides to form the final config.

andreaTP
andreaTP previously approved these changes Jan 30, 2024
Copy link
Member

@andreaTP andreaTP left a comment

Choose a reason for hiding this comment

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

A couple of minor notes but LGTM! Thanks for taking care of this Eric!

docs/generateAllConfigPartial.java Show resolved Hide resolved
ui/ui-app/init-dev.sh Show resolved Hide resolved
ui/ui-app/src/services/useConfigService.ts Show resolved Hide resolved
ui/ui-app/src/services/useConfigService.ts Outdated Show resolved Hide resolved
@EricWittmann EricWittmann merged commit e1ab98e into Apicurio:main Jan 30, 2024
19 checks passed
@EricWittmann EricWittmann deleted the feat/ui-config-from-backend branch January 30, 2024 14:09
jsenko added a commit to jsenko/apicurio-registry that referenced this pull request Feb 1, 2024
jsenko added a commit that referenced this pull request Feb 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants