-
Notifications
You must be signed in to change notification settings - Fork 158
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
OpenSearch Dashboard custom branding doesn't apply to the login screen #915
Comments
[Triage] We are the OpenSearch project, we would want to keep the OpenSearch / OpenSearch Dashboard names as is. @elfisher Do you have thoughts on what kind customizations we would want to support and if we should reconsider. |
@opensearch-project/opensearch-ux Do you have thoughts on what kind customizations we would want to support and if we should reconsider keeping the UI as is? |
@peternied - I believe custom branding includes some configs you can hook onto: See this PR: opensearch-project/OpenSearch-Dashboards#826 and the relevant configs (opensearchDoashboards.branding.logo, opensearchDoashboards.branding.applicationTitle) This way the login screen can consume the custom branding config |
Thanks @kgcreative it looks like we missed adding this customization logic into the login screen, thanks for referencing that pull request. |
Relevant docs: https://opensearch.org/docs/latest/dashboards/branding/ |
,external_osdSharedDeps_React_default.a.createElement(external_osdSharedDeps_ElasticEui_["EuiText"],{size:"m",textAlign:"center"},props.config.title|| ("please login open Search Dashboard ") i want this with my custom text but when i am changing it my custom text is not visual on login page there is same text please anyone can help me change this ???? |
Is your feature request related to a problem? Please describe.
I'm trying to customize the login screen on OpenSearch, and as far as I understood it's a part of the security plugging
Describe the solution you'd like
it would be useful to consider the login screen customization as a part of custom branding features
The text was updated successfully, but these errors were encountered: