-
Notifications
You must be signed in to change notification settings - Fork 8.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
[Fleet] Add deployment details call out to integrations browse view #113285
Comments
Pinging @elastic/fleet (Team:Fleet) |
@alexfrancoeur, Does the following design iteration align to our goal? |
I can't seem to resize the popover, but this is what I was thinking. Not committed to the text changes, was just looking to make room on the top 😄 Basically, just looking to add some more context around those two buttons. But otherwise, I think the design is good. My only concern with the primary button is that it draws a lot of attention and looks like the only meaningful action on this popover. Is there a way we can make it more subtle? |
Designs look good. Only comment might be to make the text for the button a little more clear. "View deployment and Elasticsearch details"? I'm just worried folks won't know what deployment means in this context. @gchaps might have some suggestions. |
Design updated. @KOTungseth, thank you for helping us with Unified Integrations copy so far. Would you mind taking a look at this button and popover as well, and letting us know your thoughts? |
@alexfrancoeur I suspect that for Self-Managed we don't have any Elasticsearch URL to display here that we can definitely say will be publicly available, nor can we even really safely expose to the browser (cc @elastic/kibana-security can check me on this). Reason being that admins may not want the If my suspicions are correct, some ideas for how to handle Self-Managed:
|
Agreed. |
I think as a first step it's perfectly OK to hide the UI. Anyone self-managed should be vocalized the endpoint or setting it up themselves. I think it's fine to go with @joshdover's first suggestion here. |
Agreed. Given the time constraints, I'd be OK with hiding this experience in self-managed environments. I do have one question. Is there enough value in linking out to the API key management UI? If so, we could surface this CTA but would likely need to generalize the entry point description. "View Elasticsearch deployment details" to something else. |
Great 👍
my 2c: it might be confusing. It's already a bit of a disjointed experience here where we show this UI in this app without any direct connection to language clients. That said, showing the deployment details makes sense in the context of many integrations, not just language clients, whereas API keys may be less obvious since they're not referenced in our Beats docs, etc. but Elasticsearch info is. |
Yeah, I was feeling that way as well. |
For the developer experience, it is important to have key information to push data to Elasticsearch if you're not using an Elastic data shipper like agent. This is particularly helpful for language clients. As a first step to improve this experience, we plan to add a "deployment details" call out in the top of the page to simplify the back and forth between products, config files and applications.
This user experience will have three key bit of information
/app/management/security/api_keys
) and supporting documentation (link)@dborodyansky After speaking with the clients team, we agree that referencing API key management would be a useful CTA in this popup. Could you think through what this UX might look like? We'll also need to work with @gchaps and the team to make sure this pop up has the appropriate context and matches language in cloud and self managed environments. Happy to help with the details here.
While not a primary objective for this work, it may be worth considering building this component to be reusable. I could see the benefit in including this in the API management UI as well.
cc: @philkra @sethpayne @technige @thomasneirynck @clintandrewhall @joshdover @snide @mostlyjason @mriley @linyaru @thesmallestduck
The text was updated successfully, but these errors were encountered: