Kibana: [Infra + Logs UI] Add feedback link to Kibana help menu item as custom content

Created on 31 Jan 2019  路  19Comments  路  Source: elastic/kibana

Summary

Kibana are adding a global help menu item to the global header, and in this each app is able to add custom content and link to i.e. the Discuss forum for feedback (see example in issue referenced).

The PR https://github.com/elastic/kibana/pull/29664 has a summary of how to get it added in each app.

_APM example_
screenshot 2019-02-19 at 15 28 16

Tasks

  • [x] Remove the existing "Feedback" option in the local navigation (legacy from 6.x)
  • [x] Add link to the respective discuss forums for solutions - label i.e.: Provide feedback for APM
enhancement

Most helpful comment

@alvarolobato OK, I'll create another for Uptime as @Kerry350 will take on this for Infra + Logs in this ticket.

All 19 comments

Pinging @elastic/apm-ui

馃憤

PR https://github.com/elastic/kibana/pull/29664 has been merged, so it's ready for implementation.

Thoughts on adding more links to this menu?

Sure, I don't see why not - afaik there's a space for adding practically anything in below the generic docs link.

@formgeist any chance you can make a mockup of it? If we like it, we can use that mockup for the issue to all of Observability.

On second thought, it feels a little weird that we have two documentation links; one generic and one for APM specific, but that's more how the help menu item was implemented. Also, do you want to change the "Give APM feedback" title to "APM discuss" because, that obviously would have a complete different meaning to users when viewing the links?

SGTM

So just adding Migration assistant link to the popover?

screenshot 2019-02-19 at 14 04 37

Pinging @elastic/infrastructure-ui

Pinging @elastic/uptime

I've converted this issue to an Observability-wide issue to track the implementation of adding the feedback link to the global help menu.

APM merged https://github.com/elastic/kibana/pull/30918 to add both links to the help menu. @ogupte I removed you as assignee to this issue to let the rest of Observability continue to see the changes.

@alvarolobato would it be better to create separate tickets for Infra, Logs and Uptime?

(I've assigned this for the Infrastructure / Logs implementation)

Thanks @Kerry350

@alvarolobato would it be better to create separate tickets for Infra, Logs and Uptime?

@formgeist yes, separate is easier to manage :-D

@alvarolobato OK, I'll create another for Uptime as @Kerry350 will take on this for Infra + Logs in this ticket.

Merged in #32118

Was this page helpful?
0 / 5 - 0 ratings